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

Fixes #908 Remove unwanted tags after paste #919

Merged
merged 1 commit into from
Nov 21, 2018

Conversation

fortunatomaldonado
Copy link
Member

Resent PR #909
Fixed typo that prevented the fix from working correctly.

Issue

#908
When copying and pasting a link in IE11, the pasted link will contain additional tags <u><font>.

Solution

I created a check that will search for these unwanted tags. In this issue, the <u><font color=" tag will be present and will be used to identify this issue. I did not include the color since this issue may occur with different colors. If they are found, they will be replaced with an empty space making sure to remove the start and end tags. It will search through the whole string so it will remove multiple tags.

The issue only appears in IE11 and have tested also in Chrome, Firefox, and Edge.

Let me know if there are any questions or comments.
Thank you.

Please let me know if there are any questions or comments.
Thank you.

@julien
Copy link
Contributor

julien commented Nov 21, 2018

Just started reviewing :)

:octocat: Sent from GH.

@julien julien merged commit e4cb963 into liferay:master Nov 21, 2018
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