Closed
Bug 888033
Opened 11 years ago
Closed 11 years ago
Change - add a pref to disable navigation handles
Categories
(Firefox for Metro Graveyard :: General, defect, P2)
Tracking
(Not tracked)
VERIFIED
FIXED
Firefox 25
People
(Reporter: jbecerra, Assigned: jimm)
References
Details
(Whiteboard: feature=change c=tbd u=tbd p=1)
Attachments
(1 file)
2.40 KB,
patch
|
ally
:
review+
|
Details | Diff | Splinter Review |
Tested on 2013-06-27 using latest nightly.
The <- and + handles on the edges of the Firefox Metro screen can feel a little intrusive, and more often than not I find myself wanting them to be off.
Could we add an about:config pref or a toggle setting in the options flyout that would allow the user to disable these?
Updated•11 years ago
|
Blocks: metrov1defect&change
Summary: change - add a pref to disable navigation handles → Change - add a pref to disable navigation handles
Whiteboard: feature=change c=tbd u=tbd p=0
Assignee | ||
Comment 1•11 years ago
|
||
Assignee: nobody → jmathies
Attachment #768887 -
Flags: review?(ally)
Assignee | ||
Updated•11 years ago
|
Whiteboard: feature=change c=tbd u=tbd p=0 → feature=change c=tbd u=tbd p=1
Assignee | ||
Updated•11 years ago
|
OS: Mac OS X → Windows 8 Metro
Updated•11 years ago
|
Updated•11 years ago
|
Priority: -- → P2
Comment 2•11 years ago
|
||
Comment on attachment 768887 [details] [diff] [review]
fix
Review of attachment 768887 [details] [diff] [review]:
-----------------------------------------------------------------
::: browser/metro/base/content/NavButtonSlider.js
@@ +84,5 @@
> + this._back.setAttribute("hidden", true);
> + this._plus.setAttribute("hidden", true);
> + }
> + },
> +
nit: whitespace
::: browser/metro/profile/metro.js
@@ +91,5 @@
> pref("network.protocol-handler.warn-external.ms-windows-store", false);
> pref("network.protocol-handler.external.ms-windows-store", true);
>
> +// display the overlay nav buttons
> +pref("browser.display.navbuttons", true);
nit: I'm a bit worried that this name is not specific enough, there are many nav buttons in the browser afterall. However, I don't see anything that it could be confused with in classic about:config or metro's, so we can leave it if no stroke of brilliant nomenclature hits before committing.
Attachment #768887 -
Flags: review?(ally) → review+
Assignee | ||
Comment 3•11 years ago
|
||
This pref has been updated to "browser.display.overlaynavbuttons".
Comment 4•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 25
Comment 5•11 years ago
|
||
Mozilla/5.0 (Windows NT 6.2; rv:25.0) Gecko/20130715 Firefox/25.0
Verified this on latest nightly build.
The navigation handles can be disabled by setting the "browser.display.overlaynavbuttons" pref to false in about:config.
Status: RESOLVED → VERIFIED
Comment 6•11 years ago
|
||
User Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:26.0) Gecko/20100101 Firefox/26.0
Build ID: 20130816030205
Built from http://hg.mozilla.org/mozilla-central/rev/1ed5a88cd4d0
WFM
Tested on windows 8 using latest nightly for iteration-12. The navigation handles can be disabled by setting the "browser.display.overlaynavbuttons" pref to false in about:config.
Updated•10 years ago
|
OS: Windows 8 Metro → Windows 8.1
You need to log in
before you can comment on or make changes to this bug.
Description
•