775272 - Clicking on the page to the left of the options overflow menu does not close menu

Status: VERIFIED FIXED

Description

[Michael Comella (:mcomella)]

Note: Tested only on phone device.

1) Open Firefox (16 or 17).
2) Click the button to open the options overflow menu (triple vertical dots).
3) Click on the page, directly left of the options overflow menu that opened.

Expected: The menu closes.
Actual: Nothing happens.

This happens in both landscape and portrait modes. It is a small dead zone – clicking farther to the left of the menu will correctly close it.

Possible regression from bug 769237.

Tested on Galaxy Nexus, Android 4.0.4.

Comment 3

[Cristian Nicolae (:xti)]

I cannot reproduce this issue on phones, but still occurs on tablets.

Comment 4

[Michael Comella (:mcomella)]

I can still reproduce this on my Galaxy Nexus, although the issue is MUCH more pronounced on tablets.

Comment 5

[Sriram Ramasubramanian [:sriram]]

Mike! You still raise/comment on these bugs that I run away from?? :(

Comment 6

[Sriram Ramasubramanian [:sriram]]

Created attachment 668632 [details] [diff] [review]
Patch

Basically WRAP_CONTENT on width is not shrinking it. We have to give a MeasureSpec generated over an absolute width. :(

Comment 7

[Mark Finkle (:mfinkle) (use needinfo?)]

Does this have any negative affects on phones vs tablets

Comment 8

[Sriram Ramasubramanian [:sriram]]

The size of the popup is the same on phones and tablets. Hence this would fix both.

Comment 9

[Sriram Ramasubramanian [:sriram]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/e0782e7cb92a

Comment 10

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/e0782e7cb92a

Comment 11

[Ioana Chiorean]

This is not reproducible on Firefox 19 (Nightly) - New Menu, 18 (Aurora ) - New Menu and in landscape 17 (Beta )- Old Menu . 
Steps done:  
1) Click the button to open the options overflow menu (triple vertical dots).
2) Click on the page, directly left of the options overflow menu that opened.

Reproducible on 17 portrait only. As target Milestone is 18 I will set this to verified fixed. Change the status if you decide is gonna be pushed for 17 portrait too.