Skip to content

Feature: Divide player seekbar by chapters (#6270)#13413

Open
Ecomont wants to merge 12 commits intoTeamNewPipe:refactorfrom
Ecomont:feature/chapter-markers-seekbar-on-refactor
Open

Feature: Divide player seekbar by chapters (#6270)#13413
Ecomont wants to merge 12 commits intoTeamNewPipe:refactorfrom
Ecomont:feature/chapter-markers-seekbar-on-refactor

Conversation

@Ecomont
Copy link
Copy Markdown

@Ecomont Ecomont commented Apr 13, 2026

What is it?

  • Bugfix (user facing)
  • Feature (user facing) ⚠️ Your PR must target the refactor branch
  • Codebase improvement (dev facing)
  • Meta improvement to the project (dev facing)

Description of the changes in your PR

  • Implemented visual markers on the video player's seekbar (ChaptersSeekBar) to represent chapter divisions.
  • Extracted chapter timing data from the video metadata (StreamInfo) and mapped it to the seekbar's layout bounds using transparent gaps (PorterDuff.Mode.CLEAR).
  • Ensured the visual gaps dynamically update based on the total video duration and individual chapter lengths.
  • Added dynamic chapter title tooltips and haptic feedback (HapticFeedbackConstants.CLOCK_TICK) when the user scrubs past a chapter boundary, providing a seamless visual and tactile experience similar to the original YouTube app.

Before/After Screenshots/Screen Record

Before:
https://github.com/user-attachments/assets/5da1b3e1-c701-4db0-bf70-21b020d9fd71

After:
https://github.com/user-attachments/assets/80f2cf57-d500-4670-bcd5-4e45c6d4fdcf

Fixes the following issue(s)

APK testing

The APK can be found by going to the "Checks" tab below the title. On the left pane, click on "CI", scroll down to "artifacts" and click "app" to download the zip file which contains the debug APK of this PR. You can find more info and a video demonstration on this wiki page.

Due diligence

@github-actions github-actions bot added the size/medium PRs with less than 250 changed lines label Apr 13, 2026
@ShareASmile ShareASmile added feature request Issue is related to a feature in the app GUI Issue is related to the graphical user interface player Issues related to any player (main, popup and background) UX Relating to improving user experience labels Apr 13, 2026
Copy link
Copy Markdown
Member

@theimpulson theimpulson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, seems quite nice change.

Comment thread app/src/main/java/org/schabi/newpipe/views/ChaptersSeekBar.java Outdated
@theimpulson theimpulson requested review from Stypox and TobiGr April 16, 2026 04:18
@theimpulson
Copy link
Copy Markdown
Member

As this isn't compose-specific, this should be backported to dev as well to keep feature-parity.

@Ecomont Ecomont marked this pull request as draft April 16, 2026 05:45
@Ecomont Ecomont marked this pull request as ready for review April 17, 2026 05:00
@Ecomont Ecomont changed the base branch from refactor to dev April 17, 2026 05:04
@Ecomont Ecomont changed the base branch from dev to refactor April 17, 2026 05:05
@Ecomont Ecomont force-pushed the feature/chapter-markers-seekbar-on-refactor branch from 1de5c38 to 413aaf3 Compare April 17, 2026 05:07
@Ecomont Ecomont closed this Apr 17, 2026
@Ecomont Ecomont reopened this Apr 17, 2026
@Ecomont Ecomont force-pushed the feature/chapter-markers-seekbar-on-refactor branch from 413aaf3 to 27df290 Compare April 17, 2026 05:12
@github-actions github-actions bot added size/giant PRs with more than 750 changed lines and removed size/medium PRs with less than 250 changed lines labels Apr 17, 2026
@Ecomont
Copy link
Copy Markdown
Author

Ecomont commented Apr 17, 2026

@theimpulson The backport for the dev branch is now ready, you can find it here #13430

I've ported the original Java implementation exactly as requested to maintain feature parity. Let me know if there's anything else needed on either of the PRs

@Ecomont Ecomont requested a review from theimpulson April 17, 2026 05:46
Comment thread app/src/main/java/org/schabi/newpipe/views/ChaptersSeekBar.kt Outdated
Comment thread app/src/main/java/org/schabi/newpipe/player/ui/PopupPlayerUi.java Outdated
Comment thread app/src/main/java/org/schabi/newpipe/player/ui/MainPlayerUi.java Outdated
Comment thread app/src/main/java/org/schabi/newpipe/views/FocusAwareSeekBar.kt Outdated
@Ecomont Ecomont requested a review from theimpulson April 19, 2026 12:57
@TobiGr
Copy link
Copy Markdown
Contributor

TobiGr commented Apr 19, 2026

Please rebase your PR

@Ecomont Ecomont force-pushed the feature/chapter-markers-seekbar-on-refactor branch from 9220114 to d520f8a Compare April 19, 2026 16:20
@Ecomont Ecomont force-pushed the feature/chapter-markers-seekbar-on-refactor branch from d520f8a to 80d0396 Compare April 19, 2026 16:54
@Ecomont
Copy link
Copy Markdown
Author

Ecomont commented Apr 19, 2026

Please rebase your PR

Done

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

Labels

feature request Issue is related to a feature in the app GUI Issue is related to the graphical user interface player Issues related to any player (main, popup and background) size/giant PRs with more than 750 changed lines UX Relating to improving user experience

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants