-
Notifications
You must be signed in to change notification settings - Fork 353
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
Would you like to get some of the features from my fork? #729
Comments
Thanks, those look useful. Some thoughts:
|
I prefer JS implementation because Safari loads all the lazy images in the background. The lazy attribute just lowers the priority.
If you click on a link on mobile (I tried on iOS), then go back, the link will be in focus and the tooltip will overlap half of the link, which is annoying because it's hard to take the focus off on mobile. |
Thanks for the feedback, I'll think about how to make good PRs. |
I made some features for myself:
Maybe something might be interesting to merge into the original repo? I could make a fully functional PR with good UI, tests, etc.
The text was updated successfully, but these errors were encountered: