Closed Bug 1708428 Opened 4 years ago Closed 3 years ago

[Proton] New Folder button (Bookmark panel) does not apply the specific proton dark theme

Categories

(Firefox :: Bookmarks & History, defect, P2)

defect
Points:
1

Tracking

()

VERIFIED FIXED
92 Branch
Tracking Status
firefox-esr78 --- unaffected
firefox89 --- wontfix
firefox90 --- wontfix
firefox91 --- wontfix
firefox92 --- verified

People

(Reporter: asoncutean, Assigned: mhowell)

References

(Blocks 2 open bugs)

Details

(Whiteboard: [proton-door-hangers] [priority:2a])

Attachments

(2 files)

Attached image screenshot issue.png (deleted) —

Affected versions

  • 89.0b5
  • 90.0a1 (2021-04-28)

Affected platforms

  • Windows 10

Steps to reproduce

  1. Set the dark theme
  2. Open any website
  3. Click the Star icon inside the URL bar
  4. Click the dropdown arrow (Show all the bookmarks folders)
  5. Highlight the New Folder button

Expected result

  • The focus highlight is dark theme specific

Actual result

  • The focus highlight is stuck with the light theme one

Regression range

  • Introduced with the Proton changes
Has STR: --- → yes

Interesting, I wonder if this is any way related to the issue I noticed in my PM review on macOS. In this panel, I see a few issues, but they all are related to parts of the panel not being dark themed, essentially. I don't think this will make a fix in 89, but I think we should try to fix this for 90.

Priority: -- → P2
Whiteboard: [proton-door-hangers] → [proton-door-hangers] [priority:2a]

That button just didn't get the Proton memo at all, huh. We can fix that.

Tyler, what were the other dark theme issues you had identified, and do you know if there are bugs filed for them? I notice that the folder tree and the tags list are not dark, but that should be a separate bug.

Assignee: nobody → mhowell
Status: NEW → ASSIGNED
Flags: needinfo?(tduzan)

These styles were copied from the popupnotification's .panel-footer > button
CSS, with a couple of adjustments which have comments here, and skipping over
the disabled styles because this button is never disabled.

Points: --- → 1
Pushed by mhowell@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/7d4b78379fda Apply Proton styling to the edit bookmark panel's New Folder button. r=desktop-theme-reviewers,harry
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 92 Branch

Since the status are different for nightly and release, what's the status for beta?
For more information, please visit auto_nag documentation.

Verified fixed with latest Nightly (2021-07-29) on Windows 10 and macOS 11.0.

Status: RESOLVED → VERIFIED
Flags: needinfo?(tduzan)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: