Closed Bug 1697062 Opened 4 years ago Closed 4 years ago

Add temporary Proton Light theme

Categories

(Firefox :: Theme, task)

task

Tracking

()

RESOLVED FIXED
88 Branch
Tracking Status
firefox88 --- fixed

People

(Reporter: ntim, Assigned: ntim)

References

(Blocks 1 open bug)

Details

(Whiteboard: [proton-foundations])

Attachments

(2 files)

+++ This bug was initially created as a clone of Bug #1695345 +++

The idea is to fork the built-in Light theme, and do development for Proton Light in that fork so as to not disrupt folks that don't have Proton enabled.

Then, when we reach a ship release vehicle, to merge those changes into the main-line Light theme, and remove the separate Proton Light theme.

Assignee: nobody → ntim.bugs
Status: NEW → ASSIGNED
Pushed by ntim.bugs@gmail.com: https://hg.mozilla.org/integration/autoland/rev/f2cbc95f6826 Add temporary Proton Light theme. r=jaws https://hg.mozilla.org/integration/autoland/rev/bf5e51c186a6 Update Proton Light theme to match specification. r=jaws
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 88 Branch

Hi Tim,

I noticed that Proton Light & Proton Dark themes are not persistent across sessions (after restart, the Default theme gets reenabled).

Also, Hamburger panel loses it's border with Proton Light theme, and it stays like these even after switching the theme back to default.

I wonder if we should fill these bugs since the work on themes is still in progress. We were actually testing different parts of Proton and we must cover Light and Dark theme.

Thank you!

Flags: needinfo?(ntim.bugs)

Regarding persistence, Proton Light/Proton Dark should be removed and replace Light/Dark when the time comes, so not sure that's too important. Not sure about the rest, ni? jaws about it.

Flags: needinfo?(ntim.bugs) → needinfo?(jaws)

Yeah I think we should wait on those since Proton Light/Proton Dark aren't normal themes right now due to how they're implemented. When we swap them over these issues should go away.

Flags: needinfo?(jaws)
Flags: qe-verify+
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: