Skip to content

Go back does not work in Liked Songs#151

Merged
Rigellute merged 1 commit into
masterfrom
unknown repository
Nov 18, 2019
Merged

Go back does not work in Liked Songs#151
Rigellute merged 1 commit into
masterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Nov 17, 2019

Go back q does not work as expected after scrolling pages in Liked Songs.
When i scrolled pages in Liked Songs and pushed q, i expected that the Library block is activated but nothing occurred.

This problem caused by calling push_navigation_stack in get_current_user_saved_tracks.
I tried to fix this problem by removing push_navigation_stack from get_current_user_saved_tracks and pushing to the stack is called first time.

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.

Ah yes, makes sense - scrolling was pushing to the stack!

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.

Are you able to clean up the commits? Given this small change, I feel this should be one commit. The many commits could be because of a merge rather than a rebase?

@ghost
Copy link
Copy Markdown
Author

ghost commented Nov 18, 2019

As you said, i merged branches to master and they contaminated it. I cleaned up unnecessary commits.

@Rigellute Rigellute merged commit ed228c5 into Rigellute:master Nov 18, 2019
@ghost ghost deleted the go-back-in-track-table branch November 19, 2019 02:36
nighi pushed a commit to nighi/spotify-tui that referenced this pull request Jul 11, 2025
…able

Go back does not work in Liked Songs
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.

1 participant