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)
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)
Steps:
- Install Multi Account Containers extension from Mozilla
- Switch Firefox to dark theme
- 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
Comment 2•1 year ago
|
||
: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 | ||
Updated•1 year ago
|
Assignee: nobody → emilio
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Flags: needinfo?(emilio)
Assignee | ||
Comment 3•1 year ago
|
||
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...
Comment 4•1 year ago
|
||
Set release status flags based on info from the regressing bug 1858711
status-firefox118:
--- → unaffected
status-firefox119:
--- → unaffected
status-firefox120:
--- → affected
status-firefox-esr115:
--- → unaffected
Updated•1 year ago
|
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
Comment 6•1 year ago
|
||
bugherder |
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.
Description
•