Skip to content

Keep playlist headers visible while scrolling long playlists#13503

Open
kenneySiu wants to merge 2 commits into
TeamNewPipe:refactorfrom
kenneySiu:13502-keep-playlist-buttons-visible
Open

Keep playlist headers visible while scrolling long playlists#13503
kenneySiu wants to merge 2 commits into
TeamNewPipe:refactorfrom
kenneySiu:13502-keep-playlist-buttons-visible

Conversation

@kenneySiu
Copy link
Copy Markdown

@kenneySiu kenneySiu commented May 15, 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

  • Changes to fragment_playlist.xml to use a LinearLayout with a
    dedicated playlist_header_container slot above the RecyclerView
  • Changes to PlaylistFragment.java to bind the header directly from
    the layout and return null from getListHeaderSupplier()

Before/After Screenshots/Screen Record

  • Before:
screen-newpipe.mp4
  • After:
screen-playlist-buttons.mp4

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 May 15, 2026
@ShareASmile ShareASmile added feature request Issue is related to a feature in the app playlist Anything to do with playlists in the app UX Relating to improving user experience labels May 15, 2026
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 playlist Anything to do with playlists in the app size/medium PRs with less than 250 changed lines UX Relating to improving user experience

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants