[PR #150] [MERGED] Added pagnation to user playlists #634

New issue

Closed

opened 2026-02-28 14:50:17 +03:00 by kerem · 0 comments

kerem commented

2026-02-28 14:50:17 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Rigellute/spotify-tui/pull/150
Author: @Peterwmoss
Created: 11/14/2019
Status: ✅ Merged
Merged: 11/18/2019
Merged by: @Rigellute

Base: master ← Head: master

📝 Commits (7)

cfe0587 Added pagnation to user playlists
fcfc85f Fixed formatting
bff381c Fixed formatting
a2e30b3 Fixed formatting
a3aac1d Selecting the highlighted song now plays the correct song according to the offset
4bac39b Using non-hardcoded value for pagnagtion
910ec19 Dont put each search res table on to nav_stack

📊 Changes

3 files changed (+36 additions, -5 deletions)

View changed files

📝 src/app.rs (+6 -2)
📝 src/handlers/playlist.rs (+1 -0)
📝 src/handlers/track_table.rs (+29 -3)

📄 Description

Added some simple pagnation to at least scroll through your playlist. I don't know if this is the best way to implement it but it seems to work fine from my own testing. Please excuse if my Rust is "rusty", it's my first Rust project and I don't know if there is need for memory optimization for this change to not leak memory.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/Rigellute/spotify-tui/pull/150 **Author:** [@Peterwmoss](https://github.com/Peterwmoss) **Created:** 11/14/2019 **Status:** ✅ Merged **Merged:** 11/18/2019 **Merged by:** [@Rigellute](https://github.com/Rigellute) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (7) - [`cfe0587`](https://github.com/Rigellute/spotify-tui/commit/cfe0587f09dfd94d0380085bdecadc248493912a) Added pagnation to user playlists - [`fcfc85f`](https://github.com/Rigellute/spotify-tui/commit/fcfc85feec0433359f620c1332c1f7e70ba6c0dd) Fixed formatting - [`bff381c`](https://github.com/Rigellute/spotify-tui/commit/bff381c62f0b42024d0016d1e7599e364ba29a12) Fixed formatting - [`a2e30b3`](https://github.com/Rigellute/spotify-tui/commit/a2e30b3e0f66fbb9a474dcc02f21bfbad9217c76) Fixed formatting - [`a3aac1d`](https://github.com/Rigellute/spotify-tui/commit/a3aac1dcb43f99f28b0150441f6eab46547a852d) Selecting the highlighted song now plays the correct song according to the offset - [`4bac39b`](https://github.com/Rigellute/spotify-tui/commit/4bac39bbf8adc1320ee3d0933a405f84668d3c76) Using non-hardcoded value for pagnagtion - [`910ec19`](https://github.com/Rigellute/spotify-tui/commit/910ec192e0e1d799e8ff818384d370d2a3ea2877) Dont put each search res table on to nav_stack ### 📊 Changes **3 files changed** (+36 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `src/app.rs` (+6 -2) 📝 `src/handlers/playlist.rs` (+1 -0) 📝 `src/handlers/track_table.rs` (+29 -3) </details> ### 📄 Description Added some simple pagnation to at least scroll through your playlist. I don't know if this is the best way to implement it but it seems to work fine from my own testing. Please excuse if my Rust is "rusty", it's my first Rust project and I don't know if there is need for memory optimization for this change to not leak memory. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>