-
-
Notifications
You must be signed in to change notification settings - Fork 148
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
Can Kiwix remember user settings of Dark theme? #612
Comments
Hi @alextooter, it should be able to. Which platform are you using, so I can troubleshoot? |
I am using macOS 10.15.4,Firefox 74.0.1,kiwix 2.7.0 |
OK, thanks, @alextooter . I presume that's in the Firefox extension? |
@alextooter I reproduce the bug in Firefox Extension on Windows 10. However, it is not just with this feature, it is with any feature that stores its settings in a cookie. Something must have been deprecated. Very sorry not to have picked this up in testing. @mossroy can you reproduce? In the Firefox extension, try changing any setting in Config that should persist between tab sessions, and they are lost the next time the Kiwix JS tab is opened. |
Looking at https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/Work_with_the_Cookies_API, I see we should have a We don't have this. |
I reproduce this issue with Firefox 74 (on Ubuntu), not on Chromium 80. I also reproduce on Firefox 68.6 ESR (on Windows 8.1).
in manifest.json does not help : it seems to be a permission to access cookies from other tabs/websites |
I suggest a possible solution to this in #613 . |
Based on https://bugzilla.mozilla.org/show_bug.cgi?id=1418961#c6 , it seems to be the expected behavior in Firefox extensions :
So we'll have to find another way to store the user preferences. I'll have a look at #613 |
OK. It seems this is since 55/56! I guess our users generally didn't change the defaults, then! |
It's not that surprising : in Firefox extension, you can not use SW mode, so this setting can not be changed. And the theme and cache settings are recent. So the only setting that could be affected since a long time is the "Permanently hide active content warning" |
@alextooter We are about to merge into master a fix for this issue that works in development testing on Firefox extension and that we very much hope will work in production. Please note that you will have to wait for a new release, which may take some time as we usually accumulate a number of features before making a new release. |
I like the Dark theme in v2.7,it's beautiful.After I close Kiwix page,then reopen it,Kiwix not remember my choose of Dark theme.
Can we store this option in somewhere,don't let me choose everytime?
The text was updated successfully, but these errors were encountered: