Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevent extra tab from adding in history #9039

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

ritikraj26
Copy link
Contributor

Motivation and context

This PR fixes an issue where navigating to task analytics caused #overview to be appended to the URL, requiring two back button presses to return to the previous page.
The fix ensures that the hash change does not create a new history entry, fixing navigation.

How has this been tested?

  • Manually tested in a local development environment.
  • Verified that opening task analytics no longer creates an extra history entry.
  • Ensured the browser back button correctly navigates to the previous page in a single press.

Checklist

  • I submit my changes into the develop branch
  • I have created a changelog fragment
    - [ ] I have updated the documentation accordingly
    - [ ] I have added tests to cover my changes
  • I have linked related issues (see GitHub docs)

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.

@ritikraj26 ritikraj26 requested a review from bsekachev as a code owner February 3, 2025 08:36
Copy link

sonarqubecloud bot commented Feb 3, 2025

@klakhov
Copy link
Contributor

klakhov commented Feb 4, 2025

The patch doesnt really work. Assuming I've added one more tab in analytics, I can not switch to it. It automatically goes back to first one:
history-doesntwork

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants