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

Feature Request: Option for Auto-refresh on extension settings change #92

Open
chmichael opened this issue Sep 6, 2024 · 5 comments
Open
Labels
enhancement New feature or request

Comments

@chmichael
Copy link

Hello,
Please add a "refresh button" and also when we a exclude a site it should auto-refresh.

Thank you

@chmichael chmichael changed the title Refresh button and after exluding REQUEST: Refresh button and after exluding Sep 6, 2024
@Vintagemotors
Copy link
Collaborator

I don't see any benefit to having a duplicate to the browser's existing refresh button + hotkey (f5). Adding a global auto-refresh would clear forms on a site if a user made a change to the extension setting while working on something - which would not be a good experience. There could be an option to auto-refresh but it would have to be off by default, a tooltip telling the user to refresh for changes may be an idea but I would assume the user understands the page has to be refreshed given the extension is advertised as working prior to the page being rendered.

@chmichael
Copy link
Author

Ok, an auto-refresh disabled in the settings and a notice/tooltip will be sufficient.

@Vintagemotors
Copy link
Collaborator

@ThomazPom Thoughts? I personally don't feel it is needed or should be prioritized over the existing feature request queue.

@chmichael
Copy link
Author

Dark reader is applying instantly the on/off - include/exclude site.

@ThomazPom
Copy link
Owner

ThomazPom commented Sep 18, 2024

Ultimadark’s extensive code editing on webpages prevents applying a reverse function. The only way to get the unedited version is by refreshing the page. I like the idea of adding an "Auto-refresh" option, where the page auto-refreshes if the user enables it when he excludes the website only on the current page.

@Vintagemotors About priority, we could prioritize this feature since it improves user experience and takes less than 30 minutes to implement. It's better to focus on small engacging features rather than delaying them for larger projects (like the Advanced Settings page) that require extensive development time.

Ideally, I should address bugs from critical to low and then prioritize features from shortest to longest to implement. However, constantly focusing on bug tracking and fixes sometimes makes me need a break and i do so by tackle some features. It helps me gain both a better user experience and perspective.

That's why I think prioritizing small features like this alongside bug fixes is beneficial without diving into time-consuming, larger features.

@Vintagemotors Vintagemotors added the enhancement New feature or request label Sep 24, 2024
@Vintagemotors Vintagemotors changed the title REQUEST: Refresh button and after exluding Feature Request: Option for Auto-refresh on extension settings change Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants