Skip to content

Conversation

@pmilano1
Copy link
Owner

@pmilano1 pmilano1 commented Jan 5, 2026

Summary

Enhances the media gallery with a full-screen viewer and improved navigation experience.

Changes

MediaDetailModal - Full-screen viewer

  • Full-screen dark overlay with toolbar at top
  • Keyboard navigation: arrow keys for prev/next, Escape to close
  • Zoom controls for images (50% - 300%)
  • Embedded PDF viewer with native browser support
  • Sliding info panel from right with metadata and tagging
  • Media counter showing current position (e.g., "3 / 12")

MediaCard - Tagged people display

  • Shows badges for tagged people on thumbnail hover
  • Limited to 3 visible tags with "+N more" indicator
  • Current person highlighted with primary color

Navigation

  • Previous/next buttons on full-screen modal
  • Pass allMedia array and onNavigate callback from galleries
  • Works in both PersonMediaGallery and MediaGalleryPage

Other

  • Added fileInfo translation key
  • Updated PageHeader with consistent styling

- Add badge showing count of other people tagged in media items
- Display tagged people in lightbox with clickable links
- Highlight current person in tagged people list
- Different language for photos vs documents
- Fix dataloader tests to use correct mock data structure
- Add comprehensive documentation in docs/MEDIA-TAGGING.md
- Add PersonMediaGallery component for person page media display
- Add MediaCard component with tagged people avatars
- Add MediaUploader component for file uploads
- Update MediaDetailModal to show tagged people
- Add MEDIA_FIELDS fragment with tagged_people support
- Update Media type to include tagged_people array
- Refactor media components into components/media/ directory
- Remove deprecated MediaGallery, MediaFilters, MediaGrid components
The storage_path in the database contains the full S3 URI (s3://bucket-name/path),
but the media proxy API expects just the path. This fix extracts the path portion
for proper URL generation.
- Add full-screen modal with keyboard navigation (arrow keys, escape)
- Add embedded PDF viewer support
- Add zoom controls for images
- Add sliding info panel with metadata and tagging
- Add previous/next navigation between media items
- Show tagged people badges on MediaCard thumbnails
- Update PageHeader with consistent styling
@pmilano1 pmilano1 merged commit 0cd5acc into main Jan 5, 2026
10 checks passed
@pmilano1 pmilano1 deleted the feat/media-tagged-people-display branch January 5, 2026 19:48
pmilano1 pushed a commit that referenced this pull request Jan 5, 2026
# [1.117.0](v1.116.1...v1.117.0) (2026-01-05)

### Features

* enhance media gallery with full-screen viewer and navigation ([#512](#512)) ([0cd5acc](0cd5acc))
@pmilano1
Copy link
Owner Author

pmilano1 commented Jan 5, 2026

🎉 This PR is included in version 1.117.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants