All Astronauts Posted July 9, 2023 Share Posted July 9, 2023 Bit of a deep cut this, and probably been there forever, but it would be nice to get this patched, if not in the 4x line, then 5x. _showImage method, line 305 (or approx) in ips.ui.lightbox.js // Build meta info if( image.meta ){ pieces.metaPanel .show() .html( ips.templates.render('core.lightbox.meta', { title: image.largeImage } ) ); } else { pieces.metaPanel.hide(); } Any alphanumeric/variable we pass in a front-end template through the data-ipsLightbox-meta bit, (e.g. data-ipsLightbox-meta="DERP"), will never display as you have the title variable in the render call hardcoded to image.largeImage instead of image.meta. image.largeImage is just the url to the image; once patched we can throw titles, descriptions, whatever at this. Loads better than staring at a full url. Easy fix to boot. SeNioR- 1 Link to comment Share on other sites More sharing options...
Marc Posted July 10, 2023 Share Posted July 10, 2023 Thank you for bringing this issue to our attention! I can confirm this should be further reviewed and I have logged an internal bug report for our development team to investigate and address as necessary, in a future maintenance release. Link to comment Share on other sites More sharing options...
Solution Marc Posted September 6, 2023 Solution Share Posted September 6, 2023 This issue was resolved in the recently released 4.7.13 release. Please update in order to fix this issue, and if you see any problem after that point, please let us know. Link to comment Share on other sites More sharing options...
Recommended Posts