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

Clear the cache if switching between light and dark mode #21069

Closed
szaimen opened this issue May 22, 2020 · 2 comments
Closed

Clear the cache if switching between light and dark mode #21069

szaimen opened this issue May 22, 2020 · 2 comments
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap enhancement feature: accessibility

Comments

@szaimen
Copy link
Contributor

szaimen commented May 22, 2020

How to use GitHub

  • Please use the 👍 reaction to show that you are interested into the same feature.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

Is your feature request related to a problem? Please describe.
While switching between light and dark mode, I noticed on NC18.0.4 that on chromium based browsers some files and pictures get so havily cached, that I have to manually clear the cache so that the dark mode/light mode works as expected after switching

Describe the solution you'd like
Would be really nice if Nextcloud could clear the cache after switching between light and dark mode

@szaimen szaimen added enhancement 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels May 22, 2020
@paulijar
Copy link
Contributor

At least the variable OCA.Accessibility.theme is not updated after changing between light/dark modes, until the user makes a manual refresh.

On Firefox, it is enough to hit the 'Refresh' button to fix the situation, but on Chrome/Chromium, the default behavior of the 'Refresh' button is to make a "milder refresh". There, the situation can be fixed be making the "forced refresh" by holding shift while pressing the 'Refresh' button. On the mobile Chrome, on the other hand, I haven't figured out any other way to do this except manually clearing the cached data via the History view.

@szaimen
Copy link
Contributor Author

szaimen commented Jun 11, 2021

I just tested again and couldn't reproduce the issue on NC21.0.1 anymore. If this is still happening please make sure to upgrade to the latest version. After that, feel free to reopen. The latest version can be seen by checking https://nextcloud.com/install/#install-clients and https://nextcloud.com/install/#instructions-server respectively.

@szaimen szaimen closed this as completed Jun 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap enhancement feature: accessibility
Projects
None yet
Development

No branches or pull requests

3 participants