Open
Bug 1542026
Opened 6 years ago
Updated 2 years ago
`browser.theme.reset()` applies broken version of default theme when in system dark mode
Categories
(WebExtensions :: Themes, defect, P3)
WebExtensions
Themes
Tracking
(Not tracked)
NEW
People
(Reporter: ntim, Unassigned)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
473.89 KB,
image/png
|
Details |
No description provided.
Reporter | ||
Comment 1•6 years ago
|
||
STR:
- Enable the default theme (if not already enabled)
- Enable the system dark mode
- Call browser.theme.reset()
In this case, before bug 1525762, it would reset to the non-dark default theme, but bug 1415267 requests to reset it to the currently enabled static theme (which would be the dark theme).
Summary: `browser.theme.reset()` applies broken version of default theme → `browser.theme.reset()` applies broken version of default theme in dark mode
Reporter | ||
Updated•6 years ago
|
Summary: `browser.theme.reset()` applies broken version of default theme in dark mode → `browser.theme.reset()` applies broken version of default theme when in system dark mode
Reporter | ||
Updated•6 years ago
|
Keywords: regression
Comment 2•6 years ago
•
|
||
You're saying it resets to the built-in dark theme? What's the broken part?
Flags: needinfo?(mdeboer)
See Also: → 1542044
Updated•6 years ago
|
Flags: needinfo?(mdeboer) → needinfo?(ntim.bugs)
Reporter | ||
Comment 3•6 years ago
|
||
(In reply to David Durst [:ddurst] from comment #2)
You're saying it resets to the built-in dark theme? What's the broken part?
The all white theme you see in attachment 9055968 [details] is what it resets to, instead of resetting to the either the built-in dark theme, either the default theme (depending on which behaviour we want).
Flags: needinfo?(ntim.bugs)
Comment 4•6 years ago
|
||
The priority flag is not set for this bug.
:mikedeboer, could you have a look please?
For more information, please visit auto_nag documentation.
Flags: needinfo?(mdeboer)
Updated•6 years ago
|
Flags: needinfo?(mdeboer)
Priority: -- → P2
Updated•4 years ago
|
Severity: normal → S3
Priority: P2 → P3
Updated•3 years ago
|
Has Regression Range: --- → yes
You need to log in
before you can comment on or make changes to this bug.
Description
•