Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
- Start
<a href="foobar">One Two Three</a>
- Edit
- Select
Three
, and typeFour
. - Expected
<a href="foobar">One Two Four</a>
- Actual
<a href="foobar">One Two F</a>our
Proposed resolution
TBD, blocked on CKEditor.
Remaining tasks
None.
User interface changes
None.
API changes
None.
Data model changes
None.
Comment | File | Size | Author |
---|---|---|---|
#6 | quick-edit-issue.gif | 3.51 MB | Bcwald |
Comments
Comment #2
Wim LeersPlease upload the screencast as an attachment to this issue. Those Dropbox links will stop working eventually.
Comment #3
Wim LeersAlso, this is not at all Quick Edit-specific. This is just CKEditor behavior.
Also, this is not at all related to
<a href="#">just empty anchor</a>
links. You can reproduce it with ANY link.This is an edge case of CKEditor's selection logic: as soon as you type one character, it considers that the end of the
<a>
's content, and hence it puts subsequent characters outside of the<a>
.Comment #4
Wim Leers<a href="foobar">One Two Three</a>
<a href="foobar">One Two Four</a>
<a href="foobar">One Two F</a>our
Comment #5
Wim LeersFor a moment, I thought it was a bug in
drupallink
(Drupal 8's own link plugin), but it's actually a bug in CKEditor's officiallink
as well, because I can easily reproduce this at http://ckeditor.com/demo also.Comment #6
Bcwald CreditAttribution: Bcwald commentedComment #7
Bcwald CreditAttribution: Bcwald commentedI too can replicate it on any link, so long as I start with the last character in the link. For example:
<a href="#">Test link</a>
adding anything after the K breaks. or if I delete a word and start at the end.
But if I were to start typing at the beginning or middle, say I add "Another" between Test and LInk, it works fine.
Comment #8
Bcwald CreditAttribution: Bcwald commentedComment #9
Wim Leers#6: Thanks!
#7: yep.
Now blocked on feedback from the CKEditor team. I pinged them.
I also cross-posted this to Backdrop's issue queue: https://github.com/backdrop/backdrop-issues/issues/1631 (we have a great working relationship with them wrt CKEditor-related issues).
Comment #10
Wim LeersComment #11
mlewand CreditAttribution: mlewand commentedThat's interesting couple of notes:
So it should be reported to Chromium team. It's to be determined whether it's a WebKit issue, if so we should also report it to their bugzilla.
Comment #12
Wim LeersYep, also happens in WebKit. At least in Safari 9.0.3.
Could you report this to both browser vendors? You have much more experience with that.
Comment #13
Bcwald CreditAttribution: Bcwald commentedConfirmed. It is happening on WebKit mac for me. Chrome/Safari.
I also confirmed this issue does *not* happen on FF.
Appears you are correct that it's a Webkit specific issue.
Comment #14
Wim LeersComment #15
mlewand CreditAttribution: mlewand commentedThere you go, reported to both Chromium and Webkit teams.
Comment #16
Wim LeersThanks!
Comment #17
Wim LeersAdding
prefix so it's visually clear when looking at the list of open issues which ones we cannot actually fix.Comment #29
quietone CreditAttribution: quietone at PreviousNext commentedCKEditor has been removed from core, CKEditor 4 is removed from Drupal Core in 10.0.0