Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Flyout/Dropdown light dismiss not working with Titlebar click #2028

Closed
mdtauk opened this issue Jul 19, 2019 · 3 comments · Fixed by #2438
Closed

Flyout/Dropdown light dismiss not working with Titlebar click #2028

mdtauk opened this issue Jul 19, 2019 · 3 comments · Fixed by #2438
Assignees
Labels
Area-User Interface Issues pertaining to the user interface of the Console or Terminal Issue-Bug It either shouldn't be doing this or needs an investigation. Product-Terminal The new Windows Terminal. Resolution-Fix-Committed Fix is checked in, but it might be 3-4 weeks until a release.

Comments

@mdtauk
Copy link

mdtauk commented Jul 19, 2019

Environment

Windows build number: 18362.263
Windows Terminal version (if applicable):0.0.1.0 - Built 01:02 Fri 19 July

Steps to reproduce

When you tap on the drop down button, and bring the flyout on screen. Clicking on the Titlebar will not trigger the light dismiss behaviour and the flyout remains open. Tapping on the terminal display, tab or tab bar, works as expected.

Expected behavior

Tapping on the titlebar should dismiss the flyout
image

Actual behavior

Tapping the Titlebar does not dismiss it, and if you drag the window, the flyout lags in moving to the correct position

image

@ghost ghost added Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting Needs-Tag-Fix Doesn't match tag requirements labels Jul 19, 2019
@DHowett-MSFT
Copy link
Contributor

Alas, these are both real issues and you're going to tell me to file them on the WinUI 3.0 team and I'm going to tell you that we report to the same manager, so I'll just comment all that right now and leave this open/marked.

@DHowett-MSFT DHowett-MSFT added Area-User Interface Issues pertaining to the user interface of the Console or Terminal Issue-Bug It either shouldn't be doing this or needs an investigation. Product-Terminal The new Windows Terminal. and removed Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting labels Jul 20, 2019
@ghost ghost removed the Needs-Tag-Fix Doesn't match tag requirements label Jul 20, 2019
@DHowett-MSFT DHowett-MSFT added this to the Terminal Backlog milestone Jul 20, 2019
@DHowett-MSFT DHowett-MSFT changed the title Bug Report Flyout/Dropdown light dismiss not working with Titlebar click Flyout/Dropdown light dismiss not working with Titlebar click Jul 20, 2019
@mdtauk
Copy link
Author

mdtauk commented Jul 20, 2019

Sorry @DHowett-MSFT :)

@zadjii-msft zadjii-msft self-assigned this Aug 14, 2019
@ghost ghost added the In-PR This issue has a related PR label Aug 15, 2019
@ghost ghost closed this as completed in #2438 Aug 16, 2019
ghost pushed a commit that referenced this issue Aug 16, 2019
* When the titlebar is clicked, dismiss the new tab flyout

  Fixes #2028.

* Fix this for the base IslandWindow as well
@ghost ghost added Resolution-Fix-Committed Fix is checked in, but it might be 3-4 weeks until a release. and removed In-PR This issue has a related PR labels Aug 16, 2019
@ghost
Copy link

ghost commented Aug 27, 2019

🎉This issue was addressed in #2438, which has now been successfully released as Windows Terminal Preview v0.4.2382.0.:tada:

Handy links:

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-User Interface Issues pertaining to the user interface of the Console or Terminal Issue-Bug It either shouldn't be doing this or needs an investigation. Product-Terminal The new Windows Terminal. Resolution-Fix-Committed Fix is checked in, but it might be 3-4 weeks until a release.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants