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

Link doesn't overwrite links in the current selection #464

Open
mlewand opened this issue Jun 5, 2017 · 0 comments
Open

Link doesn't overwrite links in the current selection #464

mlewand opened this issue Jun 5, 2017 · 0 comments
Labels
browser:chrome The issue can only be reproduced in the Chrome browser. status:confirmed An issue confirmed by the development team. type:bug A bug.

Comments

@mlewand
Copy link
Contributor

mlewand commented Jun 5, 2017

Are you reporting a feature or a bug?

Bug

Provide detailed reproduction steps (if any)

  1. Open CKEditor.
  2. Using "Source" button set the content to <p>Sample&nbsp;<a href="https://foo/bar/baz">https://foo/bar/baz</a> link.</p>.
  3. Go back to WYSIWYG mode by clicking "Source" button again.
  4. Put the caret at position Sample https://foo/bar/baz^ link.
  5. Press backspace three times so that "baz" is removed.
  6. Type "bom" (note: it gets typed as a plain text, due to Chrome's link handling).
  7. Make a following selectionSample [https://foo/bar/bom] link. using mouse.
  8. Click on link button.
  9. Set URL value to foo/bar/bom.
  10. Click OK.

Expected result

New URL gets applied to the whole "https://foo/bar/bom" text. Text contains a single a element.

Actual result

"https://foo/bar/bom" URL is applied only to "bom" part, while original link still exists. There are two a elements.

Following source is being produced:

<p>Sample&nbsp;<a href="https://foo/bar/baz">https://foo/bar/</a><a href="https://foo/bar/bom">bom</a>&nbsp;link.</p>

Other details

  • Browser: Chrome

Example screencast at https://www.useloom.com/share/d48c0987970d4339bd7bb9ad28cd6567

@mlewand mlewand added browser:chrome The issue can only be reproduced in the Chrome browser. status:confirmed An issue confirmed by the development team. type:bug A bug. labels Jun 5, 2017
@mlewand mlewand changed the title Link doesn't overwrite links in current seleciton Link doesn't overwrite links in the current selection Jun 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
browser:chrome The issue can only be reproduced in the Chrome browser. status:confirmed An issue confirmed by the development team. type:bug A bug.
Projects
None yet
Development

No branches or pull requests

1 participant