Skip to content
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

[BUG] protected pages cannot be disabled when granted permission #3548

Closed
erkinalp opened this issue Aug 29, 2020 · 8 comments · Fixed by #3549
Closed

[BUG] protected pages cannot be disabled when granted permission #3548

erkinalp opened this issue Aug 29, 2020 · 8 comments · Fixed by #3549
Labels

Comments

@erkinalp
Copy link
Contributor

erkinalp commented Aug 29, 2020

Describe the bug

If a protected page is allowed to be modified by darkreader, that page cannot be toggled

Expected behavior

If you grant permission for darkreader to work on protected pages, the dark mode should be able to be toggled just like another website.

Screenshots

Dark mode cannot be disabled on addons.mozilla.org (recommendation of Dark Reader is by pure chance):
image
I tried screenshotting the settings window but it disappears when I capture the picture.

System Information:

@erkinalp erkinalp added the bug label Aug 29, 2020
@Gusted
Copy link
Contributor

Gusted commented Aug 29, 2020

Hi @erkinalp!

By any chance you had a look over at #3426
Also, it should just toggle(after you refresh the UI it's a current bug that it will render the old view)

Regards,
Gusted

@erkinalp
Copy link
Contributor Author

I did. Even closing&reopening the tab did not change the result, though.

@Gusted
Copy link
Contributor

Gusted commented Aug 29, 2020

@erkinalp
As it seems you have know how to install snapshots. I don't need some fancy instructions
support.zip
Go to the master branch, fetch and pull the latest source code. Unzip support.zip.
git apply support.patch
and than you can build dark reader and it should be fixed.

Gusted added a commit that referenced this issue Aug 29, 2020
- Real-time site-toggle disable according isProtected logic and `enableForProtectedPages` option.
- tab.isProtected should not use `enableForProtectedPages` option as it doesn't get up-to-date data and will return the old value.
- Resolves #3548
Gusted added a commit that referenced this issue Sep 1, 2020
- Real-time site-toggle disable according isProtected logic and `enableForProtectedPages` option.
- tab.isProtected should not use `enableForProtectedPages` option as it doesn't get up-to-date data and will return the old value.
- Resolves #3548
@DUOLabs333
Copy link

I still have this issue: I cleared extensions.webextensions.restrictedDomains, but Dark Reader still won't work on the page. I'm on 4.9.51.

@Gusted
Copy link
Contributor

Gusted commented Jun 17, 2022

I still have this issue: I cleared extensions.webextensions.restrictedDomains, but Dark Reader still won't work on the page. I'm on 4.9.51.

Did you enable the corresponding setting in Dark Reader? By-Default dark reader won't try to modify protected site(to avoid errors )

@DUOLabs333
Copy link

What setting?

@Gusted
Copy link
Contributor

Gusted commented Jun 18, 2022

Click Dark Reader icon.
Click Dev tools (in the bottom-right corner).
Click Preview new design.
Settings -> Site list -> Enable on restricted pages.

@DUOLabs333
Copy link

Oh, it's on the new design. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants