Closed
Bug 701992
Opened 13 years ago
Closed 13 years ago
Element FullScreen API should hide Sidebar
Categories
(Firefox :: General, defect)
Firefox
General
Tracking
()
RESOLVED
FIXED
Firefox 11
People
(Reporter: alice0775, Assigned: cpearce)
References
Details
Attachments
(2 files, 4 obsolete files)
|
537 bytes,
text/html
|
Details | |
|
1.19 KB,
patch
|
dao
:
review+
|
Details | Diff | Splinter Review |
Build Identifier: http://hg.mozilla.org/mozilla-central/rev/1374294a6119 Mozilla/5.0 (Windows NT 6.1; WOW64; rv:11.0a1) Gecko/20111111 Firefox/11.0a1 ID:20111111031514 Element FullScreen API should hide Sidebar Reproducible: Always Steps to Reproduce: 1 Start Firefox with clean profile 2 Open sample html 3 Open Sidebar (Ctrl+B) 4 Click "full" button Actual Results: Sidebar would not be hidden. Expected Results: Sidebar shoould be hidden.
|
Assignee | |
Comment 1•13 years ago
|
||
Hide the sidebar when entering DOM full-screen mode.
Assignee: nobody → chris
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #577823 -
Flags: review?(dao)
|
||
Comment 2•13 years ago
|
||
Comment on attachment 577823 [details] [diff] [review] Patch v1 patch doesn't apply on mozilla-central tip
Attachment #577823 -
Flags: review?(dao)
|
|
Assignee | |
Comment 3•13 years ago
|
||
Rebased.
Attachment #577823 -
Attachment is obsolete: true
Attachment #578049 -
Flags: review?(dao)
|
|
||
Comment 4•13 years ago
|
||
Comment on attachment 578049 [details] [diff] [review] Patch v1 rebased This seems to do strange things when the sidebar isn't actually open... Warning: Empty string passed to getElementById(). Source File: chrome://browser/content/browser.xul Line: 0 Error: sidebarBroadcaster is null Source File: chrome://browser/content/browser.js Line: 8656
Attachment #578049 -
Flags: review?(dao) → review-
|
|
Assignee | |
Comment 5•13 years ago
|
||
Attachment #578049 -
Attachment is obsolete: true
Attachment #578955 -
Flags: review?(dao)
|
|
||
Comment 6•13 years ago
|
||
Like the first patch, this one doesn't apply cleanly again.
Component: Toolbars → General
QA Contact: toolbars → general
|
|
Assignee | |
Comment 7•13 years ago
|
||
Attachment #578955 -
Attachment is obsolete: true
Attachment #578955 -
Flags: review?(dao)
Attachment #578966 -
Flags: review?(dao)
|
|
||
Comment 8•13 years ago
|
||
Comment on attachment 578966 [details] [diff] [review] Patch v2 rebased This leaves the side bar splitter on screen (and trying to reopen the side bar just hides the splitter)
Attachment #578966 -
Flags: review?(dao) → review-
|
|
Assignee | |
Comment 9•13 years ago
|
||
Attachment #578966 -
Attachment is obsolete: true
Attachment #579195 -
Flags: review?(dao)
|
|
||
Updated•13 years ago
|
Attachment #579195 -
Flags: review?(dao) → review+
|
|
||
Updated•13 years ago
|
OS: Windows 7 → All
Hardware: x86 → All
|
|
Assignee | |
Comment 10•13 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/21150b624407
Target Milestone: --- → Firefox 11
|
||
Comment 11•13 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/21150b624407
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
|
||
Comment 12•13 years ago
|
||
I have created bug 701992 since the sidebar is not restored when exiting fullscreen. I am not sure if this is intentional, as seems to be the case with the find bar here http://mxr.mozilla.org/mozilla-central/source/browser/base/content/browser.js#3959
|
|
||
Comment 13•13 years ago
|
||
copy paste error, I created Bug 714675
You need to log in
before you can comment on or make changes to this bug.
Description
•