I am working on [XWIKI-19480] Disable Lightbox for profile, activity stream and livetable - XWiki.org JIRA , which was initially about some livetable loading bars and the avatar being displayed too many times inside the lightbox, but the title was modified to be more inclusive.
So as part of it, I wanted to disable the lightbox completely for livetable (which would mean that the same should be done for livedata as well) but after a discussion with Marius, the conclusion was that it might still contain images that the user would want to see in full screen and completly disabling it is not necessarly a clear solution.
To comply to this, the lightbox could be disabled only for specific parts (i.e. xwiki-livetable-pagination for livetable, image icons for livetable/livedata)
But this will lead to other problems, since when you go to the next livetable page, the new images will not be included inside the lightbox (lightbox listens to
xwiki:dom:updated, but it is not trigerred in this case). Actually now the livedata images are not included at all inside the lightbox since it’s content is loaded too late. (I am opening a lightbox issue about the fact that it’s not considering images added after, I’ll link it here)
My question related to this problems is if you think the lightbox should be disabled or not completely for livetable/livedata. The options are:
- Disable the lightbox functionality for them
- Disable the lightbox only for the code parts that generate non-content images (xwiki-livetable-pagination for livetable, image icons for livetable/livedata) and add a fix for the images loaded after (maybe listen on livetable/livedata events for updates)
- Disable the default content lightbox for livetable/livedata and use a different lightbox instance for them, to handle their specific cases (this could be usefull for the attachments tab as well)
On my side, I think the third option would be the best, to be able to handle livetable / livedata specific cases separately, like being able to use this functionality outside the xwiki content.
Related to option 2, I think that having a livedata with many images displayed between other content images could lead to confusion.