-
Notifications
You must be signed in to change notification settings - Fork 173
Description
I'm using the option fitView: true because I don't need vertical scrolling of tall images in the lightbox. There's a minor visual annoyance though: on desktop, at least in Firefox and Chrome in Windows, a disabled scrollbar remains visible even though it does nothing. And this places the close button and the next slide button right up to that disabled scrollbar, with no spacing. This makes those buttons less visible, and it makes the right side look different from the left. It's less clean than I'd like.
I've solved this by adding the following to my stylesheet:
.vbox-container { overflow-y: hidden; }
This seems to work fine on desktop and mobile.
Should this be default for cases where fitView is used? Or are there edge cases where this isn't safe, even with fitView active? If there are no issues with this tweak, then maybe consider adding this to the project.