[PR #580] [MERGED] Don't add analysis view to stack if already in it. #902

New issue

Closed

opened 2026-02-28 14:53:46 +03:00 by kerem · 0 comments

kerem commented

2026-02-28 14:53:46 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Rigellute/spotify-tui/pull/580
Author: @ericonr
Created: 9/19/2020
Status: ✅ Merged
Merged: 9/22/2020
Merged by: @Rigellute

Base: master ← Head: analysis

📝 Commits (1)

9dfd21c Don't add analysis view to stack if already in it.

📊 Changes

1 file changed (+5 additions, -3 deletions)

View changed files

📝 src/app.rs (+5 -3)

📄 Description

Before this commit, if a user pressed v multiple times, they would
have to press q the same number of times to get to the previous view.

This commit makes it so that an analysis view entry is only pushed into
the navigation stack if the current view isn't an analysis one.

_{🔄 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/580 **Author:** [@ericonr](https://github.com/ericonr) **Created:** 9/19/2020 **Status:** ✅ Merged **Merged:** 9/22/2020 **Merged by:** [@Rigellute](https://github.com/Rigellute) **Base:** `master` ← **Head:** `analysis` --- ### 📝 Commits (1) - [`9dfd21c`](https://github.com/Rigellute/spotify-tui/commit/9dfd21cb2adc0d2c556c7d1aa6efe6666c167c02) Don't add analysis view to stack if already in it. ### 📊 Changes **1 file changed** (+5 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `src/app.rs` (+5 -3) </details> ### 📄 Description Before this commit, if a user pressed `v` multiple times, they would have to press `q` the same number of times to get to the previous view. This commit makes it so that an analysis view entry is only pushed into the navigation stack if the current view isn't an analysis one. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>