Bug 89515 - Menus do not close after menu item activated
Summary: Menus do not close after menu item activated
Status: RESOLVED DUPLICATE of bug 97665
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: UI (show other bugs)
Version:
(earliest affected)
4.4.0.3 release
Hardware: Other All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected, regression
: 92681 (view as bug list)
Depends on:
Blocks:
 
Reported: 2015-02-21 01:56 UTC by Matthew Francis
Modified: 2016-02-15 12:12 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:

Attachments
Screen capture of issue (1.54 MB, video/quicktime)
2015-02-21 01:57 UTC, Matthew Francis 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Matthew Francis 2015-02-21 01:56:10 UTC 
Observed on Ubuntu 14.04 (lxde), LO 4.5 master
Not reproduced on 4.4.0.3 release

Steps to reproduce
1. Open a blank Writer document
2. Alt-i, x, i, return

Expected result
- The indexes dialog is open and the menu is not

Actual result
- The menu remains open
Comment 1 Matthew Francis 2015-02-21 01:57:10 UTC 
Created attachment 113565 [details]
Screen capture of issue
Comment 2 Matthew Francis 2015-02-21 05:23:49 UTC 
(Taking the liberty of setting this to NEW myself as a commit has been identified)
This was a bear to bisect because it's not easy to reproduce reliably, but the below commit seems to be where the issue started. Unsure why it's not apparently reproducible in 4.4.0.3 when the issue is still present on current master. Could be some subtle timing dependency?

Adding Cc: to bjoern.michaelsen@canonical.com; Could you possibly take a look at this? Thanks


commit eda624641b34a7d4315388c8ec1aebe44f63982e
Author: Bjoern Michaelsen <bjoern.michaelsen@canonical.com>
Date:   Sun May 18 14:28:39 2014 +0200

    lp#1296715: refresh invalidated menus
    
    - so we need to be a StatusListener in framework after all
    - we ware updating all menus for now, instead of just one
    - this would have a hugh performance hit when there is much change to the menu
    - thus we just invalidate the menu and update with all changes after 100ms once
Comment 3 Matthew Francis 2015-05-20 06:15:04 UTC 
Updated summary - this doesn't only happen using the keyboard, but can with the mouse as well. For the mouse, it's harder to reproduce, but fairly reliable if only the final activation is mouse based; i.e. for the original reproduction steps

- Alt-i, x (to open Insert - Indexes and Tables)
- Click on "Indexes and Tables..." using the mouse
Comment 4 Björn Michaelsen 2015-05-26 16:44:33 UTC 
Tested here with LibreOffice 4.2.8 as provided by Ubuntu/Unity --> cant reproduce.
Tested here with master@1925a57dee73933ffbdb3b57349b757470a8218a on non-Ubuntu/non-Unity -> cant reproduce after 10 attempts

Mouse interaction also works.
Comment 5 Maxim Monastirsky 2015-09-04 09:39:01 UTC 
*** Bug 92681 has been marked as a duplicate of this bug. ***
Comment 6 Maxim Monastirsky 2015-09-04 09:40:06 UTC 
We have a duplicate -> NEW
Comment 7 Robinson Tryon (qubit) 2015-12-13 11:11:59 UTC 
Migrating Whiteboard tags to Keywords: (bibisected)
[NinjaEdit]
Comment 8 Maxim Monastirsky 2016-02-15 12:12:10 UTC 

*** This bug has been marked as a duplicate of bug 97665 ***