-
Notifications
You must be signed in to change notification settings - Fork 711
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
Proposal: Click on cascading menu items to instantly open the cascading menu #939
Comments
@Poopooracoocoo, I don't think I'm understanding the proposal. Can you point me to a specific example of an app whose experience will be improved by making this change? |
There's a small (but noticeable) delay between when you hover over a parent menu item. In Win32 menus if you click the menu would open immediately, in XAML it doesn't. I agree we should do this for consistency. Should be a pretty straightforward change once we're able to make changes in WinUI 3.0. |
This comment was marked as off-topic.
This comment was marked as off-topic.
No worries! It's borderline for me too, but since there's very strong prior art in this area I think it's easy to follow suit. |
This comment was marked as spam.
This comment was marked as spam.
This issue is stale because it has been open 180 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
This comment was marked as off-topic.
This comment was marked as off-topic.
This feels beyond 'slow' in Windows App SDK v1.4.2, including in the WinUI3 gallery; it is taking 2-3 seconds of hover to open with no visual feedback, so I thought cascading menus were completely broken. Can be reproduced with the 'cascading menus' example on the menu flyouts page of WinUI3 gallery. |
@fredemmott I see no 2-3 second delay. In fact, the menu opens so quickly on hover that I don't think I can even click before the animation has finished. Do you have another machine to test on? |
I'm seeing this on two machines with Windows 10, z790, 13900k, and RTX4090 All 3 machines are running Win 10 22H2 1904.3693 with all windows updates, and the 546.17 nvidia driver |
I think this might be related to: @ranjeshj because I don't know who else to mention regarding how to further investigate this. |
Unless/until someone from Microsoft recommends a better way, my typical approach to diagnosing different behaviors on different machines is to try to find differences and eliminate them one at a time. Painful, but sometimes gets it done. Maybe check monitor setup including resolution, display scaling, text scaling. Kill all 3rd party programs, in particular anything that uses drivers or monitors other running software (antivirus, work activity monitoring, Grammarly, etc.). For reference, I'm on W10 with a 2700K and GTX 580 (lul). |
This appears to be a problem with high refresh rates: slow machines:
Fast machine:
All:
With my limited options (screenshot):
|
@fredemmott Nice catch with the refresh rate. I agree with the other comments here - it's so laggy at 2-3 seconds that you think it's broken. Setting my screen refresh rate to 60 Hz makes the pop up much faster. |
@fredemmott Nice find! Yes, there is an issue with variable refresh rate displays and how WinUI 3 keeps track of time that causes multi-second delays in DispatcherTimers firing. This is tracked by #9371, and we have a fix coming in 1.5.2. This issue (and #8921) tracks the slight delay in opening nested menus with a mouse click, and this delay exists regardless of the type of device running the app. |
@gegao18 to be clear, this is at high refresh rates - enabling/disabling variable refresh rate does no difference |
This was fixed internally and will be available in WASDK 1.7 |
Proposal: We should be able to click on cascading menu items to instantly open the cascading menu
Summary
We should be able to click on cascading menu items to instantly open the cascading menu.
Rationale
Scope
Important Notes
The animation makes menus feel slow. This is something to consider but is worth its own issue.
The text was updated successfully, but these errors were encountered: