mirror of
https://github.com/Rigellute/spotify-tui.git
synced 2026-04-26 08:05:57 +03:00
[PR #184] [MERGED] Remove focus on input when jumping back #657
Labels
No labels
bug
enhancement
good first issue
help wanted
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/spotify-tui#657
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/Rigellute/spotify-tui/pull/184
Author: @nilsrauch
Created: 12/2/2019
Status: ✅ Merged
Merged: 12/23/2019
Merged by: @Rigellute
Base:
master← Head:no-input-focus-on-back📝 Commits (1)
bd40482Remove focus on input when jumping back📊 Changes
1 file changed (+8 additions, -2 deletions)
View changed files
📝
src/main.rs(+8 -2)📄 Description
This is fixing #174
First of all I wanted to say that I'm pretty new to Rust so if you notice anything and have tips for me please feel free to share them with me :)
Also I realized that if the first action is to use the input field (thus the second route in the
navigation_stackbeing a route with the idSearch) theHomeroute will not be shown on navigating back because the second route will be popped in the match I've added and theis_none()check will break after...I didn't know how to prevent this effectively without slaughtering the code. So any ideas would be really appreciated.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.