Closed Bug 1859137 Opened 1 year ago Closed 1 year ago

Multi Account container address bar button is too dark with Firefox dark theme

Categories

(Firefox :: Theme, defect)

Desktop
All
defect

Tracking

()

RESOLVED FIXED
120 Branch
Tracking Status
firefox-esr115 --- unaffected
firefox118 --- unaffected
firefox119 --- unaffected
firefox120 --- fixed

People

(Reporter: bbhtt.zn0i8, Assigned: emilio)

References

(Regression)

Details

(Keywords: regression)

Attachments

(2 files)

Attached image image.png

Steps:

  1. Install Multi Account Containers extension from Mozilla
  2. Switch Firefox to dark theme
  3. See that the bookmark button is too dark (see image)

Previously it was white I think

$ mozregression --good 2023-10-12 --bad 2023-10-14

 8:20.68 INFO: Narrowed integration regression window from [055864c7, 4cd17f7d] (3 builds) to [055864c7, 518d9d15] (2 builds) (~1 steps left)
 8:20.68 INFO: No more integration revisions, bisection finished.
 8:20.68 INFO: Last good revision: 055864c76b58044394c7ebe5bdab6d538d29e62f
 8:20.68 INFO: First bad revision: 518d9d15677fd5f154216762af1d5867c17a90a8
 8:20.68 INFO: Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=055864c76b58044394c7ebe5bdab6d538d29e62f&tochange=518d9d15677fd5f154216762af1d5867c17a90a8
Has STR: --- → yes
Keywords: regression
Regressed by: 1858711

:emilio, since you are the author of the regressor, bug 1858711, could you take a look? Also, could you set the severity field?

For more information, please visit BugBot documentation.

Flags: needinfo?(emilio)
Assignee: nobody → emilio
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Flags: needinfo?(emilio)

These are the only ones. Whoops.

Ironically, I fixed the multi-account containers icons recently in
https://github.com/mozilla/multi-account-containers/pull/2584, but
that's still unreleased.

If we don't care about third-party pageactions we could remove this
hack once that's fixed tho...

Set release status flags based on info from the regressing bug 1858711

Attachment #9358519 - Attachment description: Bug 1859137 - Update some selectors I missed after bug 1859137. r=#theme → Bug 1859137 - Update some selectors I missed after bug 1858711. r=#theme
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0867c427ddde Update some selectors I missed after bug 1858711. r=desktop-theme-reviewers,dao
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 120 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: