[GH-ISSUE #174] Typing 'q' to exit focuses search bar #87

Closed
opened 2026-02-28 14:46:56 +03:00 by kerem · 4 comments
Owner

Originally created by @stevensonmt on GitHub (Nov 30, 2019).
Original GitHub issue: https://github.com/Rigellute/spotify-tui/issues/174

After playing some tracks, typing q to get back to the main screen and then q to quit the application puts the cursor in the search bar. Hitting 'esc' and then 'q' typically then exits the application.

Originally created by @stevensonmt on GitHub (Nov 30, 2019). Original GitHub issue: https://github.com/Rigellute/spotify-tui/issues/174 After playing some tracks, typing q to get back to the main screen and then q to quit the application puts the cursor in the search bar. Hitting 'esc' and then 'q' typically then exits the application.
kerem 2026-02-28 14:46:56 +03:00
  • closed this issue
  • added the
    bug
    label
Author
Owner

@Rigellute commented on GitHub (Dec 2, 2019):

Hi @stevensonmt, you are right. I had originally intended this to be that way, however I myself don't like it.

If anyone want to have a crack at fixing, we could possibly do this: when we are poping the navigation stack via q we can check if the poped route is Search and then immediately pop again - essentially bypassing the search route.

<!-- gh-comment-id:560301923 --> @Rigellute commented on GitHub (Dec 2, 2019): Hi @stevensonmt, you are right. I had originally intended this to be that way, however I myself don't like it. If anyone want to have a crack at fixing, we could possibly do this: when we are `pop`ing the navigation stack via `q` we can check if the `pop`ed route is `Search` and then immediately `pop` again - essentially bypassing the search route.
Author
Owner

@stevensonmt commented on GitHub (Dec 2, 2019):

I can have a go at that this evening. Seems pretty straightforward to match on the pop result of Some(Search) and pop again.

<!-- gh-comment-id:560508236 --> @stevensonmt commented on GitHub (Dec 2, 2019): I can have a go at that this evening. Seems pretty straightforward to match on the pop result of Some(Search) and pop again.
Author
Owner

@samcal commented on GitHub (Jan 4, 2020):

@Rigellute I think this issue can be closed as fixed.

<!-- gh-comment-id:570814308 --> @samcal commented on GitHub (Jan 4, 2020): @Rigellute I think this issue can be closed as fixed.
Author
Owner

@Rigellute commented on GitHub (Jan 16, 2020):

Thanks @samcal 👍

<!-- gh-comment-id:575067544 --> @Rigellute commented on GitHub (Jan 16, 2020): Thanks @samcal 👍
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/spotify-tui#87
No description provided.