Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

[QUAD] Tray Icon: Fix double click delay (way too fast / impossible to trigger) #6349

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

BenSouchet
Copy link

Changelog Description

Paragraphs contain detailed information on the changes made to the product or service, providing an in-depth description of the updates and enhancements. They can be used to explain the reasoning behind the changes, or to highlight the importance of the new features. Paragraphs can often include links to further information or support documentation.

Additional info

Paragraphs of text giving context of additional technical information or code examples.

Testing notes:

  1. start with this step
  2. follow this step

@ynbot ynbot added size/XS Denotes a PR changes 0-99 lines, ignoring general files type: bug Something isn't working labels Sep 11, 2024
@BigRoy
Copy link
Collaborator

BigRoy commented Sep 11, 2024

Delay is still the same in AYON at 100 here.

Whether anyone else feels that's too quick I'll leave up for discussion. @mkolar

@BenSouchet
Copy link
Author

Delay is still the same in AYON at 100 here.

Whether anyone else feels that's too quick I'll leave up for discussion. @mkolar

Yes and this value is way to low, following the Microsoft recommendation the correct value should be around 500 ms

https://en.wikipedia.org/wiki/Double-click#:~:text=rely%20upon%20it.-,Speed%20and%20timing,basis%20for%20other%20timed%20actions.

https://learn.microsoft.com/en-gb/windows/win32/api/winuser/nf-winuser-setdoubleclicktime?redirectedfrom=MSDN

If this value (500) is too slow, according to other projects the value shouldn't be lower than 350 ms

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
size/XS Denotes a PR changes 0-99 lines, ignoring general files type: bug Something isn't working
Projects
Status: Pending Review
Development

Successfully merging this pull request may close these issues.

3 participants