Les images de la grille Lightbox s'affichent dans le mauvais ordre

zogstrip · Mai 13, 2026, 9:16

Merci @Shauny pour le rapport. C’était un cas amusant à déboguer et à corriger

github.com/discourse/discourse

FIX: Preserve original image order in grid lightbox (#39968)

main ← fix-grid-lightbox-order

opened 09:15AM - 13 May 26 UTC

ZogStriP

+174 -3

Inside a `[grid]` block, the lightbox arrow navigation followed column-balanced …DOM order instead of the original markdown order. For four images this resulted in 1, 3, 2, 4 because `Columns` distributes items by height into the shortest column. `lightbox.js` collects items via `querySelectorAll(".lightbox")`, which returns nodes in document order — the same scrambled order the layout algorithm produced. `Columns` now tags each item with `data-lightbox-position` carrying its original index before reordering. A new `sortLightboxItems` helper in `lightbox.js` reorders items within each grid by that attribute before they reach PhotoSwipe. The existing RTL reverse continues to apply on top, so right-to-left navigation now mirrors logical order rather than the scrambled layout. https://meta.discourse.org/t/402817

cc @davidb

Sujet		Réponses	Vues
Introducing image grids in posts Announcements new-feature	42	7795	Janvier 10, 2026
Image lightboxing does not respect manual resizing or [grid] tags Bug lightbox	4	352	Novembre 30, 2024
Uploading images in order of filename Feature	30	1538	Octobre 8, 2018
Lightboxed images do not have `vertical-align` set, misaligning them with naked `img`s Bug lightbox , fixed	2	282	Juin 18, 2024
Image lightbox not working after collapsed Bug chat	4	658	Mars 17, 2023

Les images de la grille Lightbox s'affichent dans le mauvais ordre

Sujets connexes