Skip to content

ignore wysiwyg content when adding targets to outbound links#18502

Open
i-just wants to merge 1 commit into4.xfrom
bugfix/18500-outbound-links-new-windows-and-wysiwygs
Open

ignore wysiwyg content when adding targets to outbound links#18502
i-just wants to merge 1 commit into4.xfrom
bugfix/18500-outbound-links-new-windows-and-wysiwygs

Conversation

@i-just
Copy link
Contributor

@i-just i-just commented Mar 3, 2026

Description

Exclude links located in the contenteditable part of a WYSIWYG field from automatically getting the target and rel attributes.

Related issues

#18500

@i-just i-just requested a review from brandonkelly March 3, 2026 10:41
Copy link
Contributor

@7ochem 7ochem left a comment

Choose a reason for hiding this comment

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

I can confirm that this solution works. If I override /cpresources/b23bdd32/cp.js in my Developer Toolbar Sources tab and add .not('[contenteditable="true"] a'), then the issue mentioned in #18500 does not occur 👍🏻

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.

2 participants