Fix #172: Plezy Wants to "Play" Photos

[google-labs-jules]

This PR fixes issue edde746#172 where tapping on a photo would open the MediaDetailScreen with play buttons and duration, which is incorrect for photos.

Changes:

• Created lib/screens/photo_viewer_screen.dart: A new screen that displays the photo using InteractiveViewer for zooming. It supports both online (via CachedNetworkImage) and offline (via DownloadProvider and Image.file) modes.
• Modified lib/utils/media_navigation_helper.dart: Added logic to check for PlexMediaType.photo and navigate to PhotoViewerScreen instead of the default MediaDetailScreen.

This ensures that photos are displayed correctly as images, avoiding the confusion of video playback controls. Note that next/previous navigation is not yet implemented as navigateToMediaItem currently only receives a single metadata item.

---

PR created automatically by Jules for task 6079812600399769104 started by @Doezer

[google-labs-jules]

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

---

For security, I will only act on instructions from the user who triggered this task.

New to Jules? Learn more at jules.google/docs.

[Doezer]

@jules update navigation, menus... To support photo browsing

[google-labs-jules]

@jules update navigation, menus... To support photo browsing

I have updated the navigation logic to ensure that photos open in a dedicated PhotoViewerScreen instead of the video player. This screen includes a back button for proper navigation. Additionally, I've updated the context menu to hide irrelevant options like 'Mark as Watched' for photos.

[Doezer]

@jules
warning • Unused import: '../services/plex_client.dart' • lib/screens/photo_viewer_screen.dart:6:8 • unused_import
warning • The operand can't be 'null', so the condition is always 'false' • lib/screens/photo_viewer_screen.dart:61:30 • unnecessary_null_comparison
error • The named parameter 'upscale' isn't defined • lib/screens/photo_viewer_screen.dart:74:21 • undefined_named_parameter
error • The property 'grandparentTitle' can't be unconditionally accessed because the receiver can be 'null' • lib/widgets/media_context_menu.dart:190:20 • unchecked_use_of_nullable_value
error • The property 'parentTitle' can't be unconditionally accessed because the receiver can be 'null' • lib/widgets/media_context_menu.dart:195:58 • unchecked_use_of_nullable_value
error • The property 'serverId' can't be unconditionally accessed because the receiver can be 'null' • lib/widgets/media_context_menu.dart:219:39 • unchecked_use_of_nullable_value
error • The property 'ratingKey' can't be unconditionally accessed because the receiver can be 'null' • lib/widgets/media_context_menu.dart:219:60 • unchecked_use_of_nullable_value

[google-labs-jules]

@jules
warning • Unused import: '../services/plex_client.dart' • lib/screens/photo_viewer_screen.dart:6:8 • unused_import
warning • The operand can't be 'null', so the condition is always 'false' • lib/screens/photo_viewer_screen.dart:61:30 • unnecessary_null_comparison
error • The named parameter 'upscale' isn't defined • lib/screens/photo_viewer_screen.dart:74:21 • undefined_named_parameter
error • The property 'grandparentTitle' can't be unconditionally accessed because the receiver can be 'null' • lib/widgets/media_context_menu.dart:190:20 • unchecked_use_of_nullable_value
error • The property 'parentTitle' can't be unconditionally accessed because the receiver can be 'null' • lib/widgets/media_context_menu.dart:195:58 • unchecked_use_of_nullable_value
error • The property 'serverId' can't be unconditionally accessed because the receiver can be 'null' • lib/widgets/media_context_menu.dart:219:39 • unchecked_use_of_nullable_value
error • The property 'ratingKey' can't be unconditionally accessed because the receiver can be 'null' • lib/widgets/media_context_menu.dart:219:60 • unchecked_use_of_nullable_value

I have addressed the linter warnings and errors:

• Removed the unused import in photo_viewer_screen.dart.
• Removed the invalid upscale parameter from the image helper call.
• Added null-safety assertions (!) in media_context_menu.dart where required to fix unchecked usage errors.

[Doezer]

@jules warning • The operand can't be 'null', so the condition is always 'false' • lib/screens/photo_viewer_screen.dart:60:30 • unnecessary_null_comparison

[google-labs-jules]

@jules warning • The operand can't be 'null', so the condition is always 'false' • lib/screens/photo_viewer_screen.dart:60:30 • unnecessary_null_comparison

I have removed the unnecessary null comparison in lib/screens/photo_viewer_screen.dart and fixed the unchecked usage of metadata.ratingKey in lib/widgets/media_context_menu.dart.

[Doezer]

warning • The '!' will have no effect because the receiver can't be null • lib/widgets/media_context_menu.dart:219:60 • unnecessary_non_null_assertio

@jules warning • The operand can't be 'null', so the condition is always 'false' • lib/screens/photo_viewer_screen.dart:60:30 • unnecessary_null_comparison

I have removed the unnecessary null comparison in lib/screens/photo_viewer_screen.dart and fixed the unchecked usage of metadata.ratingKey in lib/widgets/media_context_menu.dart.

[google-labs-jules]

warning • The '!' will have no effect because the receiver can't be null • lib/widgets/media_context_menu.dart:219:60 • unnecessary_non_null_assertio

@jules warning • The operand can't be 'null', so the condition is always 'false' • lib/screens/photo_viewer_screen.dart:60:30 • unnecessary_null_comparison

I have removed the unnecessary null comparison in lib/screens/photo_viewer_screen.dart and fixed the unchecked usage of metadata.ratingKey in lib/widgets/media_context_menu.dart.

I have removed the unnecessary non-null assertion on metadata.ratingKey in lib/widgets/media_context_menu.dart. The previous assertion on metadata!.serverId already promotes metadata to non-nullable for the rest of the statement.