Skip to content
This repository has been archived by the owner on Nov 6, 2023. It is now read-only.

Firefox Nightly's settings dark mode #17841

Closed
rugk opened this issue Apr 27, 2019 · 11 comments
Closed

Firefox Nightly's settings dark mode #17841

rugk opened this issue Apr 27, 2019 · 11 comments

Comments

@rugk
Copy link
Contributor

rugk commented Apr 27, 2019

Type: code issue

I've discovered a big CSS problem in my add-on and checked some other add-ons, to see you are also affected.

In Firefox 68 (Nightly), the system' dark mode is governed by prefers-color-scheme is also applied to about:addons.

That includes add-on options page (presumably only if they use browserStyle?).

This is how your add-on looks like then:
image

See rugk/offline-qr-code#190 for details.


So I've opened https://discourse.mozilla.org/t/firefox-68-nightly-added-dark-version-to-about-addons-breaking-many-add-ons/38716 for discussing this with Mozilla, as I currently have no idea what their plan in this direction is.

In any case, you likely will have to adjust your add-on.

@rugk rugk changed the title Firefox Nightlies settings dark mode Firefox Nightly's settings dark mode Apr 27, 2019
@pipboy96
Copy link
Contributor

Duplicate of #12484.

@pipboy96
Copy link
Contributor

I will release a PR for this issue soon.

@rugk
Copy link
Contributor Author

rugk commented Apr 28, 2019

Okay, but remember this is currently only implemented in Nightly. So to not make it look awkward, you possibly also have to wait some time until that change hits Firefox Stable, before releasing it.
See https://bugzilla.mozilla.org/show_bug.cgi?id=1544212

@pipboy96
Copy link
Contributor

@rugk It won't do anything unless dark mode is actually on.

@rugk
Copy link
Contributor Author

rugk commented Apr 28, 2019

It will. If you use prefers-color-scheme that is supported since Firefox 67, but the dark mode in about:addons is currently just in Nightly.

@pipboy96
Copy link
Contributor

My current solution is to simply make a forced white bg on this page.

@rugk
Copy link
Contributor Author

rugk commented Apr 28, 2019

Well if you do that, then possibly you need to do nothing/adjust nothing, because maybe Firefox may do so anyway for all add-ons, see https://phabricator.services.mozilla.com/D28369.

Again: We speak about Nightly. Everything may still change.

@pipboy96
Copy link
Contributor

@rugk That's not really "handling" dark mode, more like a stopgap. I guess Nightly users should expect possible breakage, so fixing on our side immediately is not a priority, even though it's not very difficult.

@rugk
Copy link
Contributor Author

rugk commented Apr 29, 2019

That's what I've said… 😊

@pipboy96
Copy link
Contributor

@rugk Is it fine for us to not make any fixes until FF fixes that upstream?

@rugk
Copy link
Contributor Author

rugk commented Apr 29, 2019

To my judgement (I am nobody official here.) yes. When it is in beta, you should possibly have a look and tweak it, so it looks nice.

@pipboy96 pipboy96 added the hold label Apr 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants