-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Interference with other addons/settings? (privacy.resistFingerprinting) #22
Comments
Other example of non-working website (much simpler, so maybe interesting to understand what is happening): https://septatrix.github.io/cpython-dark-docs/ |
Hmm, are you really sure it is an incompatibility, i.e. with all these add-ons disabled it works? May also just depend on the website. For me it works like this:
Also note the issue #9 has been fixed in master, but not yet released on AMO. 🤔 (did not notice a difference with the site you've provided) So if it is true that other add-ons/settings cause this, could you please try to disable them one by one and test which one exactly is causing this? |
Also notice that websites could detect the presence of this add-on (due to such bugs) and at least detect whether you have a dark theme enabled, which obviously goes against the aim of |
Yes I know that this goes against If I disable |
So if I set |
"the first two" = https://kde.org, https://emojipedia.org/ ? |
No sorry, I meant in your list: https://bugzilla.mozilla.org/ and https://stuffandnonsense.co.uk. |
FYI: AFAIK, Firefox usually prevents Also an update on #22 (comment): The fix for issue #9 has now been released with v1.2 of this add-on. |
AFAICT: https://kde.org/ works (now). https://pinafore.social/ doesn't attach any "change" event listeners. https://septatrix.github.io/cpython-dark-docs/ does not work due to #24. https://stuffandnonsense.co.uk/ is currently "not working" due to the site's own fault: it specifies the color white ( I don't think this has anything to do with |
The main problem this addon has, is that it only adds css overrides extracted from media query preferred dark, but seems to be unable to remove the css from media query preferred light. See how it here says "light dark": |
That is correct. This is unfortunately a fundamental technical limitation, as explained in the Readme. |
The better API landed apparently: https://bugzilla.mozilla.org/show_bug.cgi?id=1547818#c18. |
So I have good news and bad news:
I'm sorry for that, but there is nothing I can do about that with a reasonable future support path. You can continue using v1.3 if that works for you that uses the old mechanism of manually reading and re-injecting the style-sets etc. If your are affected by this and want to have this issue fixed, please feel free to look or open a Bugzilla report and make it possible for add-ons, or the |
BTW opened #52 to show an inline-warning when this setting is enabled. |
Bug description
On my system, it works on some websites (https://bugzilla.mozilla.org/, https://stuffandnonsense.co.uk), but not on most (https://kde.org, https://emojipedia.org/, https://webkit.org/, https://pinafore.social/…).
Steps to reproduce
privacy.resistFingerprinting
(which overridesui.systemUsesDarkTheme
and makes this add-on necessary for dark theme to work at all).Actual behaviour
Most websites that are supposed don’t get dark theme.
Expected behaviour
Those websites get dark theme.
System
Operating system and version: Arch Linux
Browser and version: Firefox Nightly
Add-on version: 1.1
The text was updated successfully, but these errors were encountered: