Skip to content

Playlist: add shortcuts to jump to the start or the end#167

Merged
Rigellute merged 1 commit into
Rigellute:masterfrom
BlackYoup:end-playlist
Nov 30, 2019
Merged

Playlist: add shortcuts to jump to the start or the end#167
Rigellute merged 1 commit into
Rigellute:masterfrom
BlackYoup:end-playlist

Conversation

@BlackYoup
Copy link
Copy Markdown
Contributor

Hello,

Here's a patch to allow a user to jump to the end / start of a playlist.

I used the same shortcut as the one used to go at the end / start of the search input but I'm not sure if you are ok with that. If you are, I guess we'd have to find how to specify it in the help page.

I also didn't implement it for other variants of TrackTableContext as I'm not sure if that's worth it.

Let me know how you feel about this and thanks for your work on this client :)

Copy link
Copy Markdown
Owner

@Rigellute Rigellute left a comment

Choose a reason for hiding this comment

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

Thanks for doing this. Looks like a useful feature.

I think your proposed bindings work well. Could you add these to help.rs please?

Seems like the linter "Clippy" has some issues. Will merge once these CI problems are solved.

@BlackYoup
Copy link
Copy Markdown
Contributor Author

I think your proposed bindings work well. Could you add these to help.rs please?

Alright, will do :)

Seems like the linter "Clippy" has some issues. Will merge once these CI problems are solved.

Yes, I was waiting for your input to fix those in case you weren't ok with the changes, I'll fix it later today!

@BlackYoup BlackYoup force-pushed the end-playlist branch 3 times, most recently from 0ac167d to 7315bdb Compare November 28, 2019 12:30
Copy link
Copy Markdown
Owner

@Rigellute Rigellute left a comment

Choose a reason for hiding this comment

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

Good work @BlackYoup 👍

@Rigellute Rigellute merged commit ae47557 into Rigellute:master Nov 30, 2019
@BlackYoup BlackYoup deleted the end-playlist branch December 2, 2019 10:36
@Rigellute
Copy link
Copy Markdown
Owner

@all-contributors please add @BlackYoup for code

@allcontributors
Copy link
Copy Markdown
Contributor

@Rigellute

I've put up a pull request to add @BlackYoup! 🎉

nighi pushed a commit to nighi/spotify-tui that referenced this pull request Jul 11, 2025
Playlist: add shortcuts to jump to the start or the end
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