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.

Sign up for GitHub

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

Jump to bottom

JumpLinks - Add ability to replace the current location rather than adding to the history #11471

Open
jeff-phillips-18 opened this issue Jan 23, 2025 · 0 comments · May be fixed by #11473
Open

JumpLinks - Add ability to replace the current location rather than adding to the history #11471

jeff-phillips-18 opened this issue Jan 23, 2025 · 0 comments · May be fixed by #11473
Assignees
@jeff-phillips-18

Comments

@jeff-phillips-18
Copy link
Member

Is this a new component or an extension of an existing one?
JumpLinks component

Describe the feature
Add an option to have the navigation when a JumpLinkItem is clicked replace the current state rather than pushing the new location onto the history. Applications may not want to have the back button move back thru the jump link locations but rather return the the previous page.

Are there visuals for this feature? If applicable, please include examples for each state and for varying widths
No visual changes
@github-project-automation github-project-automation bot moved this to Needs triage in PatternFly Issues Jan 23, 2025
@jeff-phillips-18 jeff-phillips-18 linked a pull request Jan 23, 2025 that will close this issue
Open
@jeff-phillips-18 jeff-phillips-18 self-assigned this Jan 23, 2025
@thatblindgeye thatblindgeye moved this from Needs triage to PR Review in PatternFly Issues Jan 24, 2025
amorelli added a commit to amorelli/patternfly-react that referenced this issue Jan 25, 2025
@amorelli
feat(JumpLinks): support history.replaceState in JumpLinks (11471(pat…
ed9d9ba 
…ternfly#11471))

* feat(JumpLinks): add support for using history.replaceState

* docs(JumpLinks): update react-core component demo document, and react-integration demo-app demo

* test(JumpLinks): add integration tests for replaceState and pushState
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jeff-phillips-18 jeff-phillips-18

Labels
None yet
Projects
PatternFly Issues
Status: PR Review
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(JumpLinks): Add replaceNavHistory option jeff-phillips-18/patternfly-react
1 participant
@jeff-phillips-18