Skip to content

Conversation

@xsahil03x
Copy link
Member

@xsahil03x xsahil03x commented Sep 15, 2025

FLU-246

Description of the pull request

This commit introduces a fullscreen attachment gallery for viewing images and videos from feed activities.

Key changes:

  • Attachment Gallery UI:
    • Created AttachmentGallery, AttachmentGalleryItem, AttachmentGalleryHeader, AttachmentGalleryFooter, and AttachmentGalleryChrome widgets to build the gallery interface.
    • Supports image and video attachments.
    • Displays author information, timestamp, and attachment index.
    • Allows users to tap on attachments in the feed to open them in the fullscreen gallery.
  • Routing: Added a new route AttachmentGalleryRoute for the gallery.
  • Dependencies: Added chewie, photo_view, and video_player for media handling.
  • Integration: Updated UserFeedItem to navigate to the gallery when an attachment is tapped.

This commit introduces a fullscreen attachment gallery for viewing images and videos from feed activities.

Key changes:

- **Attachment Gallery UI**:
    - Created `AttachmentGallery`, `AttachmentGalleryItem`, `AttachmentGalleryHeader`, `AttachmentGalleryFooter`, and `AttachmentGalleryChrome` widgets to build the gallery interface.
    - Supports image and video attachments.
    - Displays author information, timestamp, and attachment index.
    - Allows users to tap on attachments in the feed to open them in the fullscreen gallery.
- **Routing**: Added a new route `AttachmentGalleryRoute` for the gallery.
- **Dependencies**: Added `chewie`, `photo_view`, and `video_player` for media handling.
- **Integration**: Updated `UserFeedItem` to navigate to the gallery when an attachment is tapped.
@xsahil03x xsahil03x requested a review from a team as a code owner September 15, 2025 16:33
This commit updates the `analysis_options.yaml` file to exclude generated files from analysis. This is done by adding the `analyzer.exclude` option with the pattern `lib/**/*.*.dart`.

This change is necessary because generated files often contain code that does not adhere to the project's linting rules. Excluding them from analysis prevents unnecessary warnings and errors.
Updates the Android NDK version to 27.0.12077973 and enables core library desugaring for Java 11 compatibility.

Also includes minor formatting changes in `app_router.gr.dart`.
@codecov
Copy link

codecov bot commented Sep 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 2.13%. Comparing base (eaebd26) to head (8896e85).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@          Coverage Diff          @@
##            main     #33   +/-   ##
=====================================
  Coverage   2.13%   2.13%           
=====================================
  Files        117     117           
  Lines       3140    3140           
=====================================
  Hits          67      67           
  Misses      3073    3073           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

This commit updates the Kotlin version used in the sample app's Android settings from 1.8.22 to 2.1.0.
@xsahil03x xsahil03x merged commit ff3eaa8 into main Sep 17, 2025
7 checks passed
@xsahil03x xsahil03x deleted the feat/attachment-gallery branch September 17, 2025 10:28
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.

3 participants