#3574 closed defect (fixed)
[PATCH] GUI Ordered list needs better doubleclick registration
Reported by: | elexis | Owned by: | elexis |
---|---|---|---|
Priority: | Should Have | Milestone: | Alpha 19 |
Component: | UI & Simulation | Keywords: | |
Cc: | Patch: |
Description
Since r17150 / #3557, doubleclicking is actually used in the GUI.
In both cases it is a list element, which stupidly inherits the doubleclick logic from the placeholder / parent class CGUI.cpp
. This means that whenever one doubleclicks anywhere inside the rectangle of the list, a doubleclick will be registered.
But this is not the expected behavior:
Reproduce (1): Select one item and then a different one within SELECT_DBLCLICK_RATE = 0.5
seconds. The second item will be doubleclicked instead of single-clicked, which was not intended as one didn't doubleclick that specific list item.
Reproduce (2): Doubleclick on the column header. It will reorder the list and then execute the doubleclick function.
As we don't want to hack a workaround twice in JS, we should rather make a specific logic for the COList
to register doubleclicks for list items.
Attachments (1)
Change History (8)
by , 8 years ago
Attachment: | t3574_list_item_doubleclicks_v1.patch added |
---|
comment:1 by , 8 years ago
Keywords: | patch review added |
---|---|
Milestone: | Backlog → Alpha 20 |
Summary: | GUI Ordered list needs better doubleclick registration → [PATCH] GUI Ordered list needs better doubleclick registration |
comment:2 by , 8 years ago
Milestone: | Alpha 20 → Alpha 19 |
---|
comment:4 by , 8 years ago
Owner: | set to |
---|
comment:6 by , 8 years ago
Keywords: | patch review removed |
---|
After looking at the GUI, I couldn't find a better way to fix this as elements of a list aren't actually GUI elements. Slightly annoying and I guess should be fixed someday but were that the only problem with the GUI...
Can be pushed back to A20 of course, but it would be really good to fix this in a19, as you start replays you didn't intend to start if you select multiple replays to see their details and join games which you didn't intend to join when trying to figure out who is playing on which server.