Closed Bug 655501 Opened 12 years ago Closed 12 years ago

Ctrl+Shift+Q doesn't work when a sidebar is open

Categories

(Firefox for Android Graveyard :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED
Firefox 8

People

(Reporter: linostar, Assigned: linostar)

Details

Attachments

(2 files, 2 obsolete files)

Steps to reproduce:

1- Snap right or left to show a sidebar.
2- Press Ctrl+Shift+Q.

Expected result: Portrait/Landscape mode is toggled
Actual result: View mode isn't toggled, and newtab button disappears.
Assignee: nobody → linux.anas
Status: NEW → ASSIGNED
Attached patch patch (obsolete) — Splinter Review
Attachment #536081 - Flags: review?(21)
Comment on attachment 536081 [details] [diff] [review]
patch

I don't think this works. If you are here it means the shortcuts has been caught by the chrome process already, you need to look why the content side take this shortcut and stop it instead.
Attachment #536081 - Flags: review?(21)
I tested the patch and it worked, with no side effects apparently. When one of the sidebars is open, what seems catching the keypress event is the viewport... or something else.
Attached patch patch v2 (obsolete) — Splinter Review
Attachment #536558 - Flags: review?(21)
Attachment #536081 - Attachment is obsolete: true
Attached patch patch v2Splinter Review
Attachment #536558 - Attachment is obsolete: true
Attachment #536558 - Flags: review?(21)
Attachment #536560 - Flags: review?(21)
Keywords: checkin-needed
Anas: just as a heads-up, the patch will be much easier for someone to check in correctly if you post an updated version with the author & commit-message metadata included at the top of the patch.

For details on how to do that, see:
https://developer.mozilla.org/en/Mercurial_FAQ#How_can_I_generate_a_patch_for_somebody_else_to_check-in_for_me.3f

If you don't get around to that for this bug, then you could simply post a comment here with a suggested checkin message and the author name/email you'd like to use in mozilla-central (if it's different from your bugmail address), and the person checking your patch in can add those at checkin-time.
(In reply to comment #6)
> Anas: just as a heads-up, the patch will be much easier for someone to check
> in correctly if you post an updated version with the author & commit-message
> metadata included at the top of the patch.
> 

I adjusted the patch so that it includes the metadata you mentioned.
(In reply to comment #8)
> I adjusted the patch so that it includes the metadata you mentioned.

Thanks!  For future reference, the commit message should also include the reviewer at the end (e.g. "r=vingtetun").  I added that before pushing, in this case.

Patch landed on mozilla-inbound, & it'll be merged to mozilla-central soon.
http://hg.mozilla.org/integration/mozilla-inbound/rev/efb81f7ce02a
Keywords: checkin-needed
Whiteboard: [inbound]
Merged:
http://hg.mozilla.org/mozilla-central/rev/efb81f7ce02a
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Flags: in-testsuite?
OS: Linux → All
Hardware: x86 → All
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → Firefox 8
Verified: 
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv8.0a1) Gecko/20110711 Firefox/8.0a1 Fennec/8.0a1
Device: MacBook Pro
OS: Mac OS X 10.6.7

Mozilla/5.0 (X11; Linux i686; rv8.0a1) Gecko/20110711 Firefox/8.0a1 Fennec/8.0a1
Device: MacBook Pro / VMWare
OS: Linux ubuntu 2.6.32-24-generic

Mozilla/5.0 (Windows NT 5.1; rv8.0a1) Gecko/20110711 Firefox/8.0a1 Fennec/8.0a1
Device: MacBook Pro / VMWare
OS: Windows XP
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.