View Issue Details

IDProjectCategoryView StatusLast Update
0017153MMW 5Main Panel: Toolbars & Menuspublic2021-05-19 17:46
Reporterrusty Assigned To 
PriorityurgentSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product Version5.0 
Target Version5.0.1Fixed in Version5.0.1 
Summary0017153: Menu Buttons don't deselect
DescriptionIf the user clicks a menu item (e.g. 'Play'), and then tries to de-select it by clicking 'Play' again
--> no effect (the menu just flashes/re-appears)

The user would expect that clicking the menu entry a second time would cause the menu to become deselected.

Note: this also occurs in touch mode when tapping on the left-hand panel menu. Tapping it a second time fails to deselect it (item 6 at #13150)
TagsNo tags attached.
Fixed in build2403

Relationships

related to 0014488 closedpetr Main menu random reactions to mouse click 
related to 0017887 assignedpetr Multiple-level menus don't deselect 

Activities

drakinite

2021-05-01 17:54

developer   ~0063019

I shouldn't be working on this, and I should be working on homework, but the reason this happens is because of line 379 in popupmenu.js - its mouseStateChanged handler fires before MainMenu's mousedown handler, and so it force closes the main menu. Not sure what the purpose of that line is - Commenting out the line fixes this issue, but I'm not sure if it introduces any regressions.

drakinite

2021-05-01 19:06

developer   ~0063020

Last edited: 2021-05-01 19:13

item 1) (main menu) Fixed in 2402 (I should really be working on my homework)

drakinite

2021-05-01 19:42

developer   ~0063021

Item 2) fixed in build 2402

michal

2021-05-10 15:18

developer   ~0063120

This fix caused regression. Opening main menu once causes, that navigation in lists by arrows stops working until app restart.

michal

2021-05-10 15:26

developer   ~0063121

Regression fixed in build 2403.

michal

2021-05-11 07:57

developer   ~0063129

Better fix of original issue implemented in build 2403.

rusty

2021-05-11 23:19

administrator   ~0063140

Verified 2403.