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

Version 1.15.3 #1804

Merged
merged 2 commits into from
Feb 22, 2023
Merged

Version 1.15.3 #1804

merged 2 commits into from
Feb 22, 2023

Conversation

oshi97
Copy link
Contributor

@oshi97 oshi97 commented Feb 22, 2023

Fixes

Now, when a Navigation component's parentUrl is set, clicking on the link will still
open the original url with target="_self" instead of target="_parent".

This ensures that when a link is clicked on from within an iframe, the link will only cause the iframe
to refresh, and not the parent frame.

J=SLAP-2638
TEST=manual

hook up my local SDK to a local copy of the ufc answers experience 
See that clicking navigation links works without triggering full page reloads
see that a new tab is still opened when I cmd + click
see that middle click still opens a new tab
@coveralls
Copy link

coveralls commented Feb 22, 2023

Coverage Status

Coverage: 61.439% (-0.09%) from 61.53% when pulling 0b53aac on hotfix/v1.15.3 into 85adeb0 on master.

@oshi97 oshi97 requested a review from a team February 22, 2023 16:08
Copy link
Contributor

@nmanu1 nmanu1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like the package version needs to be updated

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@oshi97 oshi97 merged commit 7526bbd into master Feb 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants