Bug #2215
open
Feature #1931: Display images
Details View: Too many images break the view
Description
For entities that have a lot of images connected to them, the image viewer breaks.
At the moment about 10 images is the maximum. It might have something to do with the "Carousel" component.
Examples can be found by navigating to a license type as those often have a lot of images associated with them, but a concrete example would be entity 20158
on the demo dev site using data from the Thanados project.
Updated by Moritz Großfurtner about 2 months ago
- Status changed from New to In Progress
It should be mostly fixed. However, for performance reasons for entities with a lot of images, e.g. the licences type, the detail view would load all the images at once, potentially slowing the site down quite a bit.
To remedy this, I would have progressively loaded the images as the user scrolls through them. In the current implementation, this leads to a rather abrupt jump to the next slide, rather than a smooth transition. Is that something we care about at this time in development or should we leave that for a separate issue? Or is it even relavant since most entities won't have that many images anyways and we can ignore the performance for those cases?
Updated by Bernhard Koschiček-Krombholz about 2 months ago
I think there will be no (non-type) entity in the near future which has this amount of images assigned to it. And I think, for Types, there is no need to show the images in a slider.
Did you test it with 50 images? How fast is it?
Updated by Moritz Großfurtner about 2 months ago
Thanks for the feedback, that makes sense. I have not yet tested it for 50 images, but I believe that should not cause any problems. I'll test it out and if everything works as expected, implement the fix without the progressive loading.
Updated by Alexander Watzinger 6 days ago
- Target version changed from 0.4.0 to 0.5.0