Closed Bug 1703957 Opened 7 months ago Closed 6 months ago

Ensure dark/light theme manifest properties match default theme dark/light mode properties for Proton

Categories

(Firefox :: Theme, task, P1)

task

Tracking

()

VERIFIED FIXED
90 Branch
Tracking Status
firefox89 --- verified
firefox90 --- verified

People

(Reporter: mconley, Assigned: jaws)

References

(Blocks 1 open bug)

Details

(Whiteboard: [proton] [proton-uplift])

Attachments

(2 files)

Unintuitively, we have both the dark theme, and the default theme with a dark mode (which is triggered by the OS). We should make sure that these two themes match before shipping Proton.

Is there a similar bug for the the light theme? The light theme and the default theme with the "light/default mode" from the OS are almost the same but there are at least two differences.

Might as well do it at the same time, I guess.

Summary: Ensure dark theme manifest properties match default theme dark mode properties for Proton → Ensure dark/light theme manifest properties match default theme dark/light mode properties for Proton
Priority: -- → P2
Whiteboard: [proton] → [proton] [priority:p2a]
Whiteboard: [proton] [priority:p2a] → [proton] [priority:2a]
Priority: P2 → P1
Whiteboard: [proton] [priority:2a] → [proton]
Assignee: nobody → jaws
Status: NEW → ASSIGNED
Pushed by jwein@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/15e19c4203af
Colors for dark/light/system/system-dark audited and confirmed consistent. r=mconley

Wiring up dependency to make it clearer what needs to land first before a beta uplift.

Depends on: 1705849
Status: ASSIGNED → RESOLVED
Closed: 6 months ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch

Backed out changeset 15e19c4203af (Bug 1703957) for causing issues with addressbar suggestions (Bug 1707257). a=backout
Backout link: https://hg.mozilla.org/mozilla-central/rev/1241823043e92c9915ed7cae9e71361c652321f4

Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Target Milestone: 90 Branch → ---
Pushed by jwein@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8fd0175a750a
Colors for dark/light/system/system-dark audited and confirmed consistent. r=mconley

Comment on attachment 9217778 [details]
Bug 1703957 - Colors for dark/light/system/system-dark audited and confirmed consistent.

Beta/Release Uplift Approval Request

  • User impact if declined: Required for MR1/Proton.
  • Is this code covered by automated tests?: No
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: Yes
  • If yes, steps to reproduce: Colors on UI elements should match those specified on https://www.figma.com/file/0XjasINHogZOxcxSmYfs4i/Colours?node-id=26%3A265
  • List of other uplifts needed: Bug 1705849
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): CSS-only changes
  • String changes made/needed: none
Attachment #9217778 - Flags: approval-mozilla-beta?
Flags: qe-verify+
Status: REOPENED → RESOLVED
Closed: 6 months ago6 months ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
QA Whiteboard: [qa-triaged]
Whiteboard: [proton] → [proton] [proton-uplift]

Comment on attachment 9217778 [details]
Bug 1703957 - Colors for dark/light/system/system-dark audited and confirmed consistent.

Approved for 89 beta 6, thanks.

Attachment #9217778 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Regressions: 1709272
Attached video Ubuntu 20 themes.webm

Verified that the themes are consistent across Windows 10 and macOS 10.15.7 with Firefox 89.0b8 and 90.0a1 and the expected colors are displayed.

Seems that on Ubuntu 20.04 the themes change a bit from default (OS) and dark/light (see attachment). Is there something to be done here?

Flags: needinfo?(jaws)

Redirecting to Harry as I'm on PTO this week.

Flags: needinfo?(jaws) → needinfo?(htwyford)

It's expected that Linux system dark and Firefox Dark will be different colors. Linux system dark uses system colors. That way, Firefox windows match other OS windows. Firefox Dark always uses the same colors regardless of system. So this is not a problem for this bug. Similarly, Linux system light and Firefox Light are expected to be slightly different.

With that said, please file bugs if QA finds any specific UI elements that are low-contrast or otherwise hard to see when a using Linux system theme.

Flags: needinfo?(htwyford)

Thank you for the response Harry. That said we can close this bug as fixed.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.