Hi devs,
I believe that we have the following issues that we could improve:
-
When using the gallery macro, you can either click on the fullscreen button or hover over the current image and click the fullscreen button for the lightbox feature. The 2 features are very close but confusing. There’s also a problem if you click on the lightbox fullscreen but: it only shows the first image of the gallery macro and not the current image, and you cannot cycle through the images.
-
Similary when in gallery fullscreen mode, you can also hover and click on the lightbox fullscreen button. This is also confusing.
-
The gallery macro currently doesn’t support displaying captions. See Loading...
Proposals:
-
Make the gallery macro depend on lightbox feature for its fullscreen button and when clicked, use the lightbox fullscreen feature. 2 sub ideas:
- If we don’t want to have the gallery macro depend on the lightbox feature, then we’d need to have both gallery and lightbox depend on the same code.
- The lightbox will display all images found on the current page. We would probably need to be able to restrain it to a list to have the same behavior as the gallery macro currently has (to only display images defined inside it).
-
If we implement 1) then 2) becomes moot (since the lightbox fullscreen will not display actions on hover)
-
The lightbox feature already supports captions so all we would have to do is have the gallery macro not remove the captions and “pass” them to the lightbox feature.
- Optional: we could also decide to display the captions in the gallery view, even when the images are not in full screen but there’s a lot less spacen so I’m not sure it’s a good idea.
WDYT?
I’m curious especially to get feedback from @mflorea and @Oana-Lavinia_Florean who coded the Gallery macro and the Lightbox feature.
Thx