Skip to content

fix: post show layout matches editor and lightbox supports navigation#23

Merged
paulocastellano merged 1 commit into
mainfrom
fix/post-show-media-order-and-lightbox
May 9, 2026
Merged

fix: post show layout matches editor and lightbox supports navigation#23
paulocastellano merged 1 commit into
mainfrom
fix/post-show-media-order-and-lightbox

Conversation

@paulocastellano
Copy link
Copy Markdown
Contributor

Summary

  • Mirror the post editor layout in the show page: media tiles on top in a 4-column grid (separate rounded cards, not joined inside a single card), caption below as plain text, labels at the bottom.
  • Replace the custom <Teleport> lightbox with the shared ImagePreviewDialog component so multi-image posts get prev/next arrows, keyboard navigation, and click-outside-to-close.

Test plan

  • Open a post with 4 images — verify the layout matches the editor (tiles on top, caption below).
  • Click any image — verify the lightbox opens with arrows (only when there are 2+ items).
  • Navigate with the arrows and with the keyboard (←/→).
  • Click outside the image — verify the lightbox closes.
  • Open a post with a single image — verify no arrows are shown.
  • Open a post with a video — verify the video plays in the lightbox.
  • Open a post with no media — verify the caption renders correctly with no spacing artifacts.

@paulocastellano paulocastellano merged commit 45b5e31 into main May 9, 2026
2 checks passed
@paulocastellano paulocastellano deleted the fix/post-show-media-order-and-lightbox branch May 9, 2026 00:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant