Invision Community 4: SEO, prepare for v5 and dormant account notifications By Matt Monday at 02:04 PM
All Astronauts Posted July 9, 2023 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
Marc Posted July 10, 2023 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.
Solution Marc Posted September 6, 2023 Solution 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.
Recommended Posts