Cmd+shift+F shortcut doesn't work while in Panorama

RESOLVED FIXED in Firefox 8

Status

Firefox Graveyard
Panorama
RESOLVED FIXED
6 years ago
a year ago

People

(Reporter: Vlad [QA], Assigned: raymondlee)

Tracking

6 Branch
Firefox 8
Dependency tree / graph

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

6 years ago
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:6.0) Gecko/20100101 Firefox/6.0 (beta1)

Cmd+shift+F shortcut doesn't work while in Panorama on MacOS X 10.7.

STR:
1. Start Firefox and hit Cmd+Shift+E to enter Panorama 
2. In Panorama, hit Cmd+Shift+F

Actual results:
Nothing happens. 

Expected results:
Firefox should enter full screen mode. If you go to Menu > View > Full Screen, it works fine.
(Assignee)

Updated

6 years ago
Depends on: 621795
(Assignee)

Comment 1

6 years ago
We only allow some of key events to work in Panorama.  Cmd+shift+F is not one of them.

I guess we should allow cmd+shift+F to work because user can go to full screen mode before entering Panorama.
(Assignee)

Updated

6 years ago
Assignee: nobody → raymond
OS: Mac OS X → All
Hardware: x86 → All
Summary: Cmd+shift+F shortcut doesn't work while in Panorama on MacOS X 10.7 → Cmd+shift+F shortcut doesn't work while in Panorama
(Assignee)

Updated

6 years ago
Depends on: 587276
(Assignee)

Comment 2

6 years ago
Created attachment 547029 [details] [diff] [review]
v1
Attachment #547029 - Flags: feedback?(tim.taubert)
Comment on attachment 547029 [details] [diff] [review]
v1

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

Thanks!
Attachment #547029 - Flags: feedback?(tim.taubert) → feedback+
(Assignee)

Updated

6 years ago
Attachment #547029 - Flags: review?(ian)
Comment on attachment 547029 [details] [diff] [review]
v1

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

Now that Tim's a peer, he can do official reviews. :)
Attachment #547029 - Flags: review?(ian) → review?(tim.taubert)
Comment on attachment 547029 [details] [diff] [review]
v1

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

Looks still good :)
Attachment #547029 - Flags: review?(tim.taubert) → review+
(Assignee)

Comment 6

6 years ago
Created attachment 548015 [details] [diff] [review]
Patch for checkin

Passed try
http://tbpl.mozilla.org/?tree=Try&rev=32fb11e9137e
Attachment #547029 - Attachment is obsolete: true
(Assignee)

Updated

6 years ago
Status: NEW → ASSIGNED
(Assignee)

Updated

6 years ago
Keywords: checkin-needed
http://hg.mozilla.org/integration/fx-team/rev/cb760f485af0
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
http://hg.mozilla.org/mozilla-central/rev/cb760f485af0
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 8
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.