-
Notifications
You must be signed in to change notification settings - Fork 414
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
Zoomable image viewer in Markdown (posts and comments) #5387
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One comment. Tested and it works great. This is really cool
e78e05b
to
0817cb8
Compare
|
@@ -128,6 +128,10 @@ | |||
} | |||
} | |||
|
|||
.img__zoomable { | |||
cursor: pointer; // 'zoom-in' would be ideal, but browser-dependant. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This could be solved down the road by adding a hover overlay to the image that indicates zoom-in.
https://stackoverflow.com/questions/21086385/how-to-make-in-css-an-overlay-over-an-image
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There may be some unintended behavior when an image is also used as a link: https://lbry.tv/@dispatch:5/markdown-tests:6
When you click the image, the link will open in a new tab, but when you return to the tab, the image is also expanded in the modal window. This could be resolved or improved at a later time though.
But overall, a very nice feature / improvement.
## Issue 4899: Ability to expand images in markdown posts for viewing
0817cb8
to
25f0e85
Compare
Issue
Closes #4899: Ability to expand images in markdown posts for viewing
Questionable
window.app.store.dispatch
andimport { doOpenModal } from 'redux/actions/app';
in a jsx file.Demo
p/s: Thanks to
btzr-io
for some pointers.