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

Replace transition with css animation #903

Open
wants to merge 3 commits into
base: gh-pages
Choose a base branch
from

Conversation

erikyo
Copy link

@erikyo erikyo commented Jun 12, 2021

This would solve a problem that can happen, especially if the images already have a transition effect applied. With animation the style is applied in a specific time frame without interfering with the css style.

it's a small modification I know, but it can make everything more beautiful (or less glitchy). Thanks for your script it's very useful to me!

This would solve a problem that can happen, especially if the images already have a transition effect applied. With animation the style is applied in a specific time frame without interfering with the css style. 

it's a small modification I know, but it can make everything more beautiful (or less glitchy). Thanks for your script it's very useful to me!
small changes to meet the standard and a fix
- missing opacity: 1 on the background while the image is loading 
- replaced spaces with tabs 
- restored the original value of .3s for the animation (personally I prefer .1s though)
@erikyo erikyo marked this pull request as ready for review June 12, 2021 17:32
this way there is even less chance of flickering for images to which opacity has been applied
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant