网格灯箱图片显示顺序错误

zogstrip · 2026年五月13日 09:16

感谢 @Shauny 的报告。这次调试和修复的过程很有趣

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

話題		回覆	觀看
Introducing image grids in posts Announcements new-feature	42	8404	2026年一月10日
Image lightboxing does not respect manual resizing or [grid] tags Bug lightbox	4	418	2024年十一月30日
Uploading images in order of filename Feature	30	1614	2018年十月8日
Lightboxed images do not have `vertical-align` set, misaligning them with naked `img`s Bug lightbox , fixed	2	316	2024年六月18日
Image lightbox not working after collapsed Bug chat	4	705	2023年三月17日

网格灯箱图片显示顺序错误

相關主題