Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Unstyled admin bar appears when browsing cached pages offline #252

Closed
westonruter opened this issue Mar 25, 2020 · 5 comments
Closed

Unstyled admin bar appears when browsing cached pages offline #252

westonruter opened this issue Mar 25, 2020 · 5 comments
Assignees
Labels
invalid This doesn't seem right service-workers

Comments

@westonruter
Copy link
Collaborator

westonruter commented Mar 25, 2020

When going offline in WordPress 5.4, the admin bar links are showing up the header like so:

image

I'm using this this Basic Site Caching plugin: https://gist.github.com/westonruter/1a63d052beb579842461f6ad837715fb

Granted, perhaps the problem is that this plugin is not adding the admin-bar.css to the runtime cache. But at the same time, perhaps the admin bar should not show up at all when serving from cache?

@westonruter
Copy link
Collaborator Author

This will be addressed by #264.

@westonruter
Copy link
Collaborator Author

Maybe to fix this we should just add /wp-admin/ to the list of routes that are cached?

@ankitrox
Copy link
Collaborator

@westonruter There are two things, I observed.

  1. admin-bar.css file is being cached in core-assets in cache storage.
  2. I am not able to reproduce the issue in WP 5.9 with Twenty Twenty theme.

If this is still the issue, can you walk me through steps to reproduce the same?

@westonruter
Copy link
Collaborator Author

westonruter commented Feb 28, 2022

  1. admin-bar.css file is being cached in core-assets in cache storage.

Oh! So what this means is that this issue is obsolete. It was filed on March 2020 before the introduction of the core asset cache as part of #338, which was merged in December 2020.

Aside: This should also be mitigated by #436, in that if a user has Offline Browsing turned off then the offline/error page should not be cached at all in that case. Actually, no as this is not about the offline page but rather cached pages.

@westonruter westonruter added the invalid This doesn't seem right label Feb 28, 2022
@westonruter westonruter removed this from the 0.7 milestone Feb 28, 2022
@pooja-muchandikar
Copy link

As per this #252 (comment) the issue seems to be invalid

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
invalid This doesn't seem right service-workers
Projects
None yet
Development

No branches or pull requests

3 participants