Enable the extensions.webextensions.themes.enabled preference to ride the trains

RESOLVED FIXED in Firefox 55

Status

defect
RESOLVED FIXED
2 years ago
Last year

People

(Reporter: jaws, Assigned: mikedeboer)

Tracking

(Blocks 1 bug)

unspecified
mozilla55
Dependency tree / graph

Firefox Tracking Flags

(firefox55 fixed)

Details

(Whiteboard: [themes][triaged])

Attachments

(1 attachment, 1 obsolete attachment)

Once we get the dependencies for bug 1330335 fixed then we can enable the API on Nightly / DevEdition.

Updated

2 years ago
Whiteboard: [themes, triaged]
Assignee

Updated

2 years ago
Depends on: 1343821
No longer depends on: 1333953
No longer depends on: themingapi-framework
webextensions: --- → ?

Updated

2 years ago
webextensions: ? → ---
Assignee: nobody → mwein

Updated

2 years ago
Duplicate of this bug: 1359260

Comment 2

2 years ago
Based on the meeting last week, we think this should be on by default and just ride to release and there was enough done to allow this. Is there anything in bug 1330335 that's a blocker?
Keywords: dev-doc-needed
Whiteboard: [themes, triaged] → [themes][triaged]
Comment hidden (mozreview-request)
(In reply to Andy McKay [:andym] from comment #2)
> Is there
> anything in bug 1330335 that's a blocker?

are we not doing bug 1344243?  :(
Assignee

Comment 5

2 years ago
(In reply to Andrew Swan [:aswan] from comment #4)
> are we not doing bug 1344243?  :(

(I added my comment to bug 1344243)
Assignee

Comment 6

2 years ago
mozreview-review
Comment on attachment 8861218 [details]
Bug 1341722 - Enable themes by default now that all of its blockers have landed

https://reviewboard.mozilla.org/r/133184/#review136160

I'm not ready to remove the pref yet. Let's add it to firefox.js as `true` instead, so that Thunderbird and Seamonkey can still opt-out (by default).
Attachment #8861218 - Flags: review?(mdeboer) → review-

Comment 7

2 years ago
Is this still something we plan on doing for 55?
Assignee

Comment 8

2 years ago
I'll write the patch today - thanks for the ping!
Flags: needinfo?(mdeboer)
Assignee

Updated

2 years ago
Assignee: mwein → mdeboer
Status: NEW → ASSIGNED
Flags: needinfo?(mdeboer)
Assignee

Updated

2 years ago
Attachment #8861218 - Attachment is obsolete: true

Comment 10

2 years ago
mozreview-review
Comment on attachment 8872956 [details]
Bug 1341722 - enable WebExtension themes by default on Nightly and Aurora channels.

https://reviewboard.mozilla.org/r/144496/#review148538

::: browser/app/profile/firefox.js:201
(Diff revision 1)
>  pref("extensions.{972ce4c6-7e08-4474-a285-3208198ce6fd}.description", "chrome://browser/locale/browser.properties");
>  
>  pref("extensions.webextensions.themes.icons.buttons", "back,forward,reload,stop,bookmark_star,bookmark_menu,downloads,home,app_menu,cut,copy,paste,new_window,new_private_window,save_page,print,history,full_screen,find,options,addons,developer,synced_tabs,open_file,sidebars,share_page,subscribe,text_encoding,email_link,forget,pocket");
> +#ifndef RELEASE_OR_BETA
> +pref("extensions.webextensions.themes.enabled", true);
> +#endif

WHy not all channels and let it ride the trains?
Comment hidden (mozreview-request)
Shouldn't the extensions.webextensions.themes.icons.enabled be enabled too? Or is this planned for later?
Assignee

Comment 13

2 years ago
(In reply to Richard Marti (:Paenglab) from comment #12)
> Shouldn't the extensions.webextensions.themes.icons.enabled be enabled too?
> Or is this planned for later?

Later and it's not even certain whether we'll enable it at all... our UX dept is not at all convinced yet that the feature is a good idea. We'll see in the near future.

Comment 14

2 years ago
mozreview-review
Comment on attachment 8872956 [details]
Bug 1341722 - enable WebExtension themes by default on Nightly and Aurora channels.

https://reviewboard.mozilla.org/r/144496/#review148960

Might want to update bug title too.
Attachment #8872956 - Flags: review?(amckay) → review+
Assignee

Updated

2 years ago
Summary: Enable the extensions.webextensions.themes.enabled preference on Nightly / DevEdition → Enable the extensions.webextensions.themes.enabled preference to ride the trains

Comment 15

2 years ago
Pushed by mdeboer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e5b66b34bd7c
enable WebExtension themes by default on Nightly and Aurora channels. r=andym

Comment 16

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/e5b66b34bd7c
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55

Updated

Last year
Product: Toolkit → WebExtensions
You need to log in before you can comment on or make changes to this bug.