[Webkit] Anchors editing broken in some cases

[Olek Nowodziński]

1. Set the following HTML

   <p>f<a id="x" name="x">o</a>o</p>
   

2. Double-click the anchor.

Expected: Existing anchor is edited.

Actual: A new anchor is edited.

​git:b5acf14a92926 is the first bad commit.

[Piotrek Koszuliński]

Not as critical as it might seem. It's only reproducible if anchor has adjacent non-white-space text nodes. Most likely doubleclick first cause expanding selection to whole word and then it's too late for anchor feature to react.

[Olek Nowodziński]

Pushed fix to ​branch:t/11822 (+tests).

This issue was much easier than suggested in comment:2. Basically ​git:b5acf14a restricted selection#selectElement( evt.data.link ) to elements to be edited in link dialog only, while non-empty anchors (empty ones are faked with img) also must be selected.

[Piotrek Koszuliński]

I haven't noticed this while reviewing related tickets in 4.4, but the logic of doubleclick listeners (0,10,20) is tangled. Am I right that image2 could listen with priority <10, and link with the default one? I would be glad if we could revert this commit ​git:b5acf14a. Is it possible?

[Olek Nowodziński]

Replying to Reinmar:

I haven't noticed this while reviewing related tickets in 4.4, but the logic of doubleclick listeners (0,10,20) is tangled. Am I right that image2 could listen with priority <10, and link with the default one? I would be glad if we could revert this commit ​git:b5acf14a. Is it possible?

Yes, it is possible. However, if Image2 listened with priority < 10 and canceled the event before it reaches listeners in link plugin, then it would need to handle ​this logic, which is what I wanted to avoid. My idea was to do it gently, so the process of link discovery (evt.data.dialog = 'link' vs 'anchor') remains on the Link plugin-side. And thanks to 0,10,20 listeners, it is possible – there is no redundant code, which would need attention, if something has changed in Link plugin. Also, if we decide to bring support for Image2+anchors in the future, there's nothing to do there, because Link plugin takes care of invoking the right dialog etc.

[Piotrek Koszuliński]

Fixed on master with ​git:70a8947 on dev and c0d14c0 on tests.