-
Notifications
You must be signed in to change notification settings - Fork 97
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
Don't show a thumbnail, when the image link is a plain text link #434
Comments
Hi, What do you expect to be shown there? Displaying full size images would be a terrible idea for performance. |
Yes, you are right, it is terrible for performance. I use Fancybox at my phpBB forum as imageviewer. There are often plaintext links to external images. The display of these links as thumbnail is already better solved in Fancybox 5 than in version 4. The display as image, as in Fancybox 3 would be nicer. |
I expect there the full size image.
https://raw.githubusercontent.com/IMC-GER/images/main/temp/FancyboxScreenshot.jpg
Am 09.03.2023 um 12:33 schrieb Jānis Skarnelis:
…
OK, but you did not answer to the question - "What do you expect to be
shown there"? Do you expect to display there full size image or some
dummy element?
—
Reply to this email directly, view it on GitHub
<#434 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AVE5P3D4UZUN67ESVUC4BQTW3G5ZFANCNFSM6AAAAAAVUFHMHQ>.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Hi, Please, upgrade to the latest, I have added image src as the fallback. |
Is your feature request related to a problem? Please describe.
Fancybox5 don't show a thumbnail, when the image link is a plain text link.
Please look at this example: https://jsfiddle.net/dvhmaL1u/
Describe the solution you'd like
PLease display the thumbnail.
Describe alternatives you've considered
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: