Skip to content

Fix settings back button navigation - #730

Open
ManorDev wants to merge 1 commit into
Polyfrost:mainfrom
ManorDev:fix/settings-back-button
Open

Fix settings back button navigation#730
ManorDev wants to merge 1 commit into
Polyfrost:mainfrom
ManorDev:fix/settings-back-button

Conversation

@ManorDev

Copy link
Copy Markdown

Fixes #715

Changes

  • Use replace() instead of push() when navigating between Settings pages so tabs don't accumulate in router history.
  • Preserve the original route for the Settings back button.
  • The Back button now returns directly to the screen from which Settings was opened.

Testing

  • Manually tested navigation through multiple Settings pages.
  • Verified Back returns directly to the original screen.
  • cargo test -p oneclient_app — 59 passed.
  • cargo clippy -p oneclient_app --all-targets — passed.
  • git diff --check — passed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

the back button in settings progresses through all previous windows instead of just taking you back to the main screen

1 participant