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

Fix overwriting of target attribute of anchors rendered by IPython.display #7215

Conversation

kavishdahekar
Copy link

Bug fix for issue where user provided value for target attribute of anchor tags rendered by IPython.display were overwritten by '_blank' or '_self'.

References

Fixes #6827

Code changes

The bugfix will overwrite the target attribute only if it was not set.
Setting of the rel attribute to noopener moved to separate if block.

User-facing changes

Users rendering anchor elements using IPython.display will be able to set custom values for "target" attribute.

Backwards-incompatible changes

@jupyterlab-dev-mode
Copy link

Thanks for making a pull request to JupyterLab!

To try out this branch on binder, follow this link: Binder

@kavishdahekar kavishdahekar changed the title Fixes jupyterlab#6827 Fixes overwriting of target attribute of anchors rendered by IPython.display Sep 16, 2019
@kavishdahekar kavishdahekar changed the title Fixes overwriting of target attribute of anchors rendered by IPython.display Fix overwriting of target attribute of anchors rendered by IPython.display Sep 16, 2019
Copy link
Member

@blink1073 blink1073 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks! The CI failure looks unrelated, so I kicked it.

@blink1073
Copy link
Member

Thanks again @kavishdahekar, and congratulations on your first JupyterLab PR!

@blink1073 blink1073 merged commit 30be9f7 into jupyterlab:master Sep 16, 2019
@kavishdahekar
Copy link
Author

Thanks @blink1073 :) Looking forward to contributing more to the project.

@jasongrout
Copy link
Contributor

@meeseeksdev backport to 1.x

meeseeksmachine pushed a commit to meeseeksmachine/jupyterlab that referenced this pull request Sep 18, 2019
jasongrout added a commit that referenced this pull request Sep 18, 2019
…5-on-1.x

Backport PR #7215 on branch 1.x (Fix overwriting of target attribute of anchors rendered by IPython.display)
@jasongrout jasongrout modified the milestones: 1.2, 2.0 Oct 11, 2019
@lock lock bot added the status:resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion. label Nov 10, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Nov 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement pkg:rendermime status:resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

IPython.display HTML is overwriting the target attribute in hyperlinks
4 participants