-
Notifications
You must be signed in to change notification settings - Fork 9
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
Blog post background flashing #99
Comments
Suggest we disable in the meantime? |
maboa
added a commit
that referenced
this issue
Jun 6, 2022
There are more in depth fixes but in the short term this flash is a liability for anybody prone to seizures. This disables our constantly updating reading time, but that's OK since it was causing harm. We'll make a longer term fix that restores the functionality at a later date. Related to Issue #99
Addressed by #108 |
Opening again because useful info here for actually fixing the problem. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Here's the problem -
biffud.com/lib/ui/templates/BlogPostTpl.js
Line 15 in 3dd3f26
Fix would be to make a subcomponent for the reading time that manages timer state by itself, rather than at the page template level where it causes the entire page to re-render.
The text was updated successfully, but these errors were encountered: