664114 - Context Menu doesn't resize properly when switching the window orientation

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Cristian Nicolae (:xti)]

Build id : Mozilla/5.0 (Android;Linux armv7l;rv:5.0)Gecko/20110613
Firefox/5.0 Fennec/5.0
Device: Motorola Droid 2
OS: Android 2.2

Steps to reproduce:
1. Open Fennec app
2. Go to http://slickdeals.net/
3. Long tap on Tap and hold down on the Slickdeals banner image link
4. Switch device to landscape

Expected result:
Context Menu resize properly due to device orientation changes at step 4.

Actual result:
The Context Menu does not resize when switching the orientation.

Comment 1

[Cristian Nicolae (:xti)]

Attachment: screenshot

Comment 2

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

What's wrong with the screen shot? It seems like the context menu has changed it's size, right?

Comment 3

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

Oh, I see what you mean. The height overflows.

Comment 4

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

Attachment: patch

I found two problems: menu overflows and context hint (the url on first row) is white-on-white

Fixes in this patch include:
* Adds the color / background-color to the right elements (context-header gets the background and share-hint / context-hint get color)
* Fixes the overflow issue by adding display: -moz-box to the context-commands richlistbox

The match syncs the contextmenu and the menulist behaviors a bit too:
* Adds flex to the container for the context container
* Makes sizeToContent use the container padding to create the width for the popup
* Makes sure sizetoContent is called before we make the panels visible, in case the richlistitems are larger than the window. We need to resize before they are visible or the UI will pan.

Comment 5

[Wesley Johnston (:wesj)]

Comment on attachment 539289 [details] [diff] [review]
patch

Review of attachment 539289 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good!

::: mobile/chrome/content/common-ui.js
@@ +1220,1 @@
>    },

Can we rename this._panel to this._container?

Comment 6

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

Attachment: patch for beta

minimal fix needed to fix the overflow in mozilla-beta

Comment 7

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

Comment on attachment 539333 [details] [diff] [review]
patch for beta

minimal beta patch. fennec-only. no nays from IRC and beta triage meeting.

Comment 8

[Philipp von Weitershausen [:philikon]]

Backed out in http://hg.mozilla.org/integration/mozilla-inbound/rev/d05a24569346 due to a persistent failure in dom/tests/mochitest/dom-level0/test_innerWidthHeight_script.html on Android opt.

Comment 9

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

(In reply to comment #8)
> Backed out in
> http://hg.mozilla.org/integration/mozilla-inbound/rev/d05a24569346 due to a
> persistent failure in
> dom/tests/mochitest/dom-level0/test_innerWidthHeight_script.html on Android
> opt.

Wes - can you think of any reason for this failure?

Comment 10

[Aaron Train [:aaronmt]]

Verifying Fixed on the beta channel
http://hg.mozilla.org/releases/mozilla-beta/rev/6eb872ce9e29

Mozilla/5.0 (Android; Linux armv7l; rv:5.0) Gecko/20110614 Firefox/5.0 Fennec/5.0

Leaving open re-landing on mozilla-inbound.

Comment 11

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

(In reply to comment #8)
> Backed out in
> http://hg.mozilla.org/integration/mozilla-inbound/rev/d05a24569346 due to a
> persistent failure in
> dom/tests/mochitest/dom-level0/test_innerWidthHeight_script.html on Android
> opt.

Philipp - I notice the test failed for your backout too. Could it be intermittent?

Comment 12

[Phil Ringnalda (:philor)]

My guess would be the other patch in the same push, http://hg.mozilla.org/integration/mozilla-inbound/rev/b9ee105d755c

And note that you don't actually have just one test failure, you have that one, plus a timeout in browser_thumbnails.js that's being called intermittent but is actually permanent, and a timeout in twinopen.

Comment 13

[Mounir Lamouri (:mounir)]

Pushed to m-c:
http://hg.mozilla.org/mozilla-central/rev/19e4a93ebff5

Comment 14

[Anna (Waverley)]

VERIFIED FIXED on:

Mozilla /5.0 (Android;Linux armv7l;rv:7.0a1) Gecko/20110621 Firefox/7.0a1 Fennec/7.0a1 

Device: HTC Desire Z(Android 2.2)

Note: "Not fixed on aurora"

Comment 15

[LegNeato]

http://hg.mozilla.org/releases/mozilla-beta/rev/bb311256b95b