Closed Bug 1379778 Opened 7 years ago Closed 7 years ago

Rename Compact Light and Compact Dark themes to Light and Dark

Categories

(Firefox :: Theme, defect)

56 Branch
All
macOS
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 1366363

People

(Reporter: Felipe, Unassigned)

Details

Attachments

(1 file)

Attached image default-vs-compact.png
The compact theme (with standard density) is actually 2px taller than the default theme on OSX. See the screenshot for a side-by-side comparison.
Now that we have separated Density from Appearance we need to rename Compact Light and Compact Dark to Light and Dark respectively.
Summary: Compact theme is 2px taller than default theme → Rename Compact Light and Compact Dark themes to Light and Dark
We already have bug 1366363 to rename the themes. However, I'm tempted to suggest that we should actually fix the bug here, probably by removing most/all of the custom styling here (that isn't just colours - though the colours could possibly be re-implemented using the new lwtheme stuff?), and migrating users to the compact UI density setting if they're using the compact lwthemes. Up to the visual theme if they want to split that work or just dupe this over - but we should really make sure this gets cleaned up a bit.
See Also: → 1366363
Whiteboard: [photon-visual][triage]
No longer blocks: photon-visual
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → DUPLICATE
See Also: 1366363
Whiteboard: [photon-visual][triage]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: