Closed Bug 1366363 Opened 4 years ago Closed 4 years ago

Rename Compact Dark and Compact Light themes to just "Dark" and "Light"

Categories

(Firefox :: Toolbars and Customization, enhancement, P1)

enhancement

Tracking

()

VERIFIED FIXED
Firefox 57
Iteration:
57.1 - Aug 15
Tracking Status
firefox57 --- verified

People

(Reporter: dao, Assigned: mkohler)

References

Details

(Whiteboard: [photon-visual][p1][57])

Attachments

(1 file)

Firefox 57 will have compact and touch modes independently from current compact themes, but we want to keep these themes as dark/light alternatives to the default. We need to rename them accordingly.
Blocks: 1366369
Priority: -- → P2
Flags: qe-verify+
QA Contact: brindusa.tot
What should this be renamed to? I'm happy to do this if you tell me the new names. :)
This will have to wait till Nightly 57.

The new names should perhaps just be "Light" and "Dark". I don't have a better idea. Will need to check with Shorlander.
Blocks: 1367284
No longer blocks: 1367284
See Also: → 1367419
See Also: 1367419
Duplicate of this bug: 1176577
Blocks: 1176577
See Also: → 1379778
See Also: 1379778
Duplicate of this bug: 1379778
Summary: Rename Compact Dark and Compact Light themes → Rename Compact Dark and Compact Light themes to just "Dark" and "Light"
I don't think we need to worry about internal IDs for now, they're not visible to users in any way.
Flags: needinfo?(dao+bmo)
Attachment #8893520 - Flags: review?(dao+bmo)
Comment on attachment 8893520 [details]
Bug 1366363 - Rename Compact Dark and Compact Light themes to just 'Dark' and 'Light'

https://reviewboard.mozilla.org/r/164608/#review169934

::: browser/components/nsBrowserGlue.js:630
(Diff revision 1)
>        LightweightThemeManager.addBuiltInTheme({
>          id: "firefox-compact-light@mozilla.org",
> -        name: gBrowserBundle.GetStringFromName("compactLightTheme.name"),
> -        description: gBrowserBundle.GetStringFromName("compactLightTheme.description"),
> +        name: gBrowserBundle.GetStringFromName("lightTheme.name"),
> +        description: gBrowserBundle.GetStringFromName("lightTheme.description"),
>          headerURL: "resource:///chrome/browser/content/browser/defaultthemes/compact.header.png",
> -        iconURL: "resource:///chrome/browser/content/browser/defaultthemes/compactlight.icon.svg",
> +        iconURL: "resource:///chrome/browser/content/browser/defaultthemes/light.icon.svg",

The iconURL change is broken because you didn't rename the SVG file itself...

You could either revert this change or rename the SVG file.
Attachment #8893520 - Flags: review?(dao+bmo)
Assignee: nobody → me
Status: NEW → ASSIGNED
Priority: P2 → P1
Comment on attachment 8893520 [details]
Bug 1366363 - Rename Compact Dark and Compact Light themes to just 'Dark' and 'Light'

https://reviewboard.mozilla.org/r/164608/#review169946

Thanks!
Attachment #8893520 - Flags: review?(dao+bmo) → review+
Comment on attachment 8893520 [details]
Bug 1366363 - Rename Compact Dark and Compact Light themes to just 'Dark' and 'Light'

https://reviewboard.mozilla.org/r/164608/#review169950
Attachment #8893520 - Flags: review+
Attachment #8893520 - Flags: review+
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3f96fc2c078c
Rename Compact Dark and Compact Light themes to just 'Dark' and 'Light' r=dao
This seems to have caused a spike in failures like https://treeherder.mozilla.org/logviewer.html#?job_id=120808860&repo=autoland

This was a previously known intermittent but starting on your push, it started failing multiple times per push.

Backed out in https://hg.mozilla.org/integration/autoland/rev/a4e153426d4c5b2b862446c1e975530e9054dc72
Flags: needinfo?(me)
Hrm, those failures are still frequently happening on the backout... I guess this can reland as-is.
Flags: needinfo?(me)
Keywords: checkin-needed
You're going to need to reopen the mozreview request if you want someone to land it for you.
Flags: needinfo?(me)
Pushed by kwierso@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/64526a900fd5
Rename Compact Dark and Compact Light themes to just 'Dark' and 'Light' r=dao
Keywords: checkin-needed
Flags: needinfo?(me)
Duplicate of this bug: 1387293
https://hg.mozilla.org/mozilla-central/rev/64526a900fd5
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
Iteration: --- → 57.1 - Aug 15
https://hg.mozilla.org/projects/date/rev/3f96fc2c078cc6b6b150bbc191dbbaf2b9f39eea
Bug 1366363 - Rename Compact Dark and Compact Light themes to just 'Dark' and 'Light' r=dao

https://hg.mozilla.org/projects/date/rev/64526a900fd51d1fb784fca7eac2ae9b0bae30d1
Bug 1366363 - Rename Compact Dark and Compact Light themes to just 'Dark' and 'Light' r=dao
Managed to reproduce the issue on an affected build. Bug 1366363 is fixed & verified.
Version : Firefox Nightly 57.0a1
build ID: 20170809100326
OS      :Windows 10 X 64 bit. 

[bugday-20170809]
I tested this issue on Mac Os X 10.12, Windows 10 x64, Ubuntu 16.04 with FF Nightly 57.0a1(2017-08-11) Compact Dark and Compact Light themes are now renamed to "Dark" and "Light". I will mark this as verified fixed.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
No longer blocks: 1176577
Blocks: 1368808
You need to log in before you can comment on or make changes to this bug.