The default bug view has changed. See this FAQ.

[tabletui] Opening a new tab in portrait mode does not open the awesomescreen

VERIFIED FIXED in Firefox 9

Status

Fennec Graveyard
General
VERIFIED FIXED
6 years ago
5 years ago

People

(Reporter: mbrubeck, Assigned: mbrubeck)

Tracking

({regression})

Firefox 9
Firefox 9
regression

Details

(Whiteboard: [fixed in aurora])

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Created attachment 563902 [details] [diff] [review]
patch

Steps to reproduce:
1. Open fennec in portrait mode on tablet
2. Tap the tabs button to open the menu
3. Press the "New tab" button in the menu

Expected results: New tab opens, urlbar is focused, and awesomescreen drop-down appears.

Actual results: New tab opens but urlbar is not focused and awesomescreen does not appear.

This is a problem from my patch in bug 688840.  hidePortraitMenu refers to an undefined variable "resizeHandler".  This moves resizeHandler back to its own method of TabsPopup, and calls bind from init to avoid the problem that bug 688840 was trying to fix.
Attachment #563902 - Flags: review?(wjohnston)
(Assignee)

Updated

6 years ago
Assignee: nobody → mbrubeck
Status: NEW → ASSIGNED
Whiteboard: [has patch]
Comment on attachment 563902 [details] [diff] [review]
patch

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

Please add some sort of comment explaining why we're doing:

this.resizeHandler = this.resizeHandler.bind(this);

and maybe some comments at the event listeners noting its already bound... Otherwise I'm sure I'll stare at it someday wondering what's going on.
Attachment #563902 - Flags: review?(wjohnston) → review+
(Assignee)

Comment 2

6 years ago
Pushed with added comment: https://hg.mozilla.org/integration/mozilla-inbound/rev/a89bf0c36977
Whiteboard: [has patch] → [pushed]
Target Milestone: --- → Firefox 10
(Assignee)

Comment 3

6 years ago
Comment on attachment 563902 [details] [diff] [review]
patch

Requesting approval for Aurora 9.  This is a fix for a regression in Firefox 9 that breaks the user experience of opening a new tab (in portrait mode, on tablets).  The fix is mobile-only and very localized.  It fixes an undefined variable access exception that happens every time the portrait tab menu is hidden.
Attachment #563902 - Flags: approval-mozilla-aurora?
https://hg.mozilla.org/mozilla-central/rev/a89bf0c36977
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Verified Fixed on Nightly
Mozilla/5.0 (Android; Linux armv7l; rv:10.0a1) Gecko/20111003 Firefox/10.0a1 Fennec/10.0a1
status-firefox10: --- → fixed

Updated

6 years ago
Attachment #563902 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
(Assignee)

Comment 6

6 years ago
https://hg.mozilla.org/releases/mozilla-aurora/rev/4a81215700fb
status-firefox9: affected → fixed
Whiteboard: [pushed] → [fixed in aurora]
Target Milestone: Firefox 10 → Firefox 9

Comment 7

6 years ago
Verified Fixed on Aurora
Mozilla/5.0 (Android; Linux armv7l; rv:9.0a2) Gecko/20111013 Firefox/9.0a2 Fennec/9.0a2

When pressing  "New Tab" button wew tab opens, urlbar is focused, and awesomescreen drop-down appears.
Status: RESOLVED → VERIFIED
This landed on aurora for 9, not tracking.
tracking-firefox9: ? → -
You need to log in before you can comment on or make changes to this bug.