Invision Community 4: SEO, prepare for v5 and dormant account notifications Matt November 11, 2024Nov 11
Posted July 9, 20231 yr 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.
July 10, 20231 yr 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.
September 6, 20231 yr Solution 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.