Skip to content

Add Enqueue option to external URL open dialog#31

Draft
christophkroeppl wants to merge 1 commit intoInfinityLoop1308:devfrom
christophkroeppl:feature/enqueue-from-open-dialog
Draft

Add Enqueue option to external URL open dialog#31
christophkroeppl wants to merge 1 commit intoInfinityLoop1308:devfrom
christophkroeppl:feature/enqueue-from-open-dialog

Conversation

@christophkroeppl
Copy link
Copy Markdown

Summary

  • Adds "Enqueue" option to the share/open dialog when opening external YouTube links
  • Only appears when a player is already active with items in queue
  • Appends the stream to current play queue without interrupting playback
  • Ported from NewPipe's RouterActivity enqueue feature

Changes

  • RouterActivity.getChoicesForService(): adds enqueue choice (conditional on queue > 0)
  • RouterActivity.FetcherService.getResultHandler(): routes enqueue to NavigationHelper.enqueueOnPlayer()
  • settings_keys.xml: adds enqueue_key string resource

Test plan

  • Open a video in PipePipe, start playing
  • Share a different YouTube link to PipePipe
  • Verify "Enqueue" option appears in the dialog
  • Tap Enqueue, verify video is added to queue without interrupting current playback
  • Verify "Enqueue" does NOT appear when no player is active

🤖 Generated with Claude Code

When a player is already active with items in queue, the share/open
dialog now shows an "Enqueue" option that appends the stream to the
current play queue without interrupting playback.

Ported from NewPipe's RouterActivity enqueue feature.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
manuel09 pushed a commit to manuel09/SuperPipeClient that referenced this pull request Apr 25, 2026
…tached in Comments (InfinityLoop1308#31)

Replace `PictureDialog` with `ImageViewerActivity`, ~borrowed from my other projects~.

Thanks to [com.github.chrisbanes:PhotoView](https://github.com/Baseflow/PhotoView), now this image detail dialog (activity) can:
- Full screen
- Zoom
- Swipe to switch
- Download / Copy URI / Open in browser

Reviewed-on: https://codeberg.org/NullPointerException/PipePipeClient/pulls/31
Co-authored-by: chr56 <chr0056@gmail.com>
Co-committed-by: chr56 <chr0056@gmail.com>
@InfinityLoop1308
Copy link
Copy Markdown
Owner

Hi is this ready for review?

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.

2 participants