Skip to content

Add missing scrollbar to Playlists tab - #223

Closed
rivaldi8 wants to merge 3 commits into
timusus:mainfrom
rivaldi8:fix/alexm-playlist-tab-missing-scrollbar
Closed

rivaldi8 wants to merge 3 commits into
timusus:mainfrom
rivaldi8:fix/alexm-playlist-tab-missing-scrollbar

Conversation

@rivaldi8

@rivaldi8 rivaldi8 commented May 1, 2026

Copy link
Copy Markdown
Contributor

The tab lost the scrollbar after the migration to Compose.

@rivaldi8
rivaldi8 force-pushed the fix/alexm-playlist-tab-missing-scrollbar branch from e48ad40 to 5640360 Compare May 1, 2026 17:59
timusus added a commit that referenced this pull request Sep 24, 2026
Every other library tab (Songs, Albums, Artists, Genres) has a fast
scroller; Playlists was missing one. Split PlaylistList's Ready-state
content into an internal composable holding LazyColumn state so a
FastScroller can be layered over it, matching GenreList.kt/SongList.kt.
Playlists have no natural sort key to show as popup text, so add a
minimal NoPopup-paired noPopupText helper to FastScroller.kt.

Changelog: Added a fast scrollbar to the Playlists tab, matching the other library tabs

Co-authored-by: Àlex Magaz Graça <alexandre.magaz@gmail.com>
@timusus

timusus commented Sep 24, 2026

Copy link
Copy Markdown
Owner

Landed on main as 6035027, reworked onto current main, with you credited as co-author. Verified by unit tests plus the full emulator suite, which now carries a check for this change. Thanks!

@timusus timusus closed this Sep 24, 2026
@rivaldi8
rivaldi8 deleted the fix/alexm-playlist-tab-missing-scrollbar branch September 26, 2026 00:49
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