Closed
Bug 657525
Opened 13 years ago
Closed 12 years ago
Site-specific preferences for chromeless app tabs
Categories
(Firefox :: Tabbed Browser, defect)
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: Felipe, Assigned: Felipe)
References
Details
Attachments
(1 file)
6.56 KB,
patch
|
Gavin
:
feedback+
|
Details | Diff | Splinter Review |
Moving the site-specific patch from bug 644721 over here. Updated to tip and to go along with the latest patch there. This adds a menu entry in app tabs that says "Show toolbars for this site" and disables the chromeless app tabs features for that domain
Assignee | ||
Updated•13 years ago
|
Attachment #532806 -
Flags: review?(gavin.sharp)
Comment 1•13 years ago
|
||
Comment on attachment 532806 [details] [diff] [review] Patch It would be better to use an actual boolean as the pref value rather than "true"/"". "Show toolbars for this site" is a bit odd. Seems like "for this pinned tab" (or "app tab" or whatever) would be perhaps clearer, since the setting only applies to that site as an app tab, rather than to that site globally (i.e. when loaded in a normal tab). Maybe that's a distinction not worth making, though. This looks fine, but will need some rebasing on top of a new patch for bug 644721, I think.
Attachment #532806 -
Flags: review?(gavin.sharp) → feedback+
Comment 2•13 years ago
|
||
This menu item should not appear when "browser.tabs.chromelessAppTabs" is false.
Comment 3•12 years ago
|
||
wontfix as per bug 769101 comment 16
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•