Magnific Popup mobile css

Johani · May 21, 2021, 1:20am

Good catch

I made a small change in core based on that.

github.com/discourse/discourse

UX: Improve positioning of controls in lightboxes (#13108)

committed 01:18AM - 21 May 21 UTC

hnb-ku

+3 -0

This commit fixes an issue where controls scroll in lightboxes with large image…s (after zooming in) Before: https://d11a6trkgmumsb.cloudfront.net/original/3X/0/5/05024730b3b877c472dff5f4d63155121d370297.mp4 Notice how controls like the close button, the next and previous button, and the image metadata also scroll? This is an undesired behavior. After: https://d11a6trkgmumsb.cloudfront.net/original/3X/8/0/8047bab7351831e9b9df8c3cf0eb368c7529ae8a.mp4 This is the desired behavior; only the image should scroll. The changes in this PR apply to both desktop and mobile.

Thanks for reporting the issue and the fix @Don

Topic		Replies	Views
Custom Lightbox Prototype dev	2	715	July 5, 2021
Custom Lightbox theme-component	3	1039	May 4, 2022
Header shifts to the right when you open image gallery ux	4	595	March 20, 2019
Allow swipe up/down actions to close images on mobile feature	8	908	October 15, 2022
Images not adjusting to the width on mobile devices support	8	628	February 4, 2022

Magnific Popup mobile css

Related Topics