Ticket #332 (closed Bug: fixed)

Opened 17 months ago

Last modified 11 months ago

Hitting Enter after links continues the link in the new paragraph

Reported by: hanki_pank Owned by: martinkou
Priority: Normal Milestone: FCKeditor 2.5 Beta
Component: UI : Enter Key Version: FCKeditor 2.4.2
Keywords: Confirmed Firefox SD-COE Cc: pkdille, Jyhem, nyloth

Description (last modified by fredck) (diff)

Handling of <a>-Tags within <li> is broken. Reproduction at the nightly build just right now :

  1. Hit return for a new line.
  2. Press "Insert Bulleted list" - Button
  3. Press "Insert Link" - Button
  4. enter www.blah.org as target and hit return to return back to gui
  5. Hit return to get new bullet.
  6. type "asd"

provides as source :

<p>This is some <strong>sample text</strong>. You are using <a href="http://www.fckeditor.net/">FCKeditor</a>.</p>
<ul>
    <li><a href="http://www.blah.org">www.blah.org</a></li>
    <li><a href="http://www.blah.org">asd</a></li>
</ul>
<p>&nbsp;</p>

Meaning, the <a>-Tag from the first bullet has been inherited by the second one, which is not what users expect, I guess.

Change History

Changed 17 months ago by hanki_pank

Here a more readable version....

Handling of <a>-Tags within <li> is broken.

Reproduction at the nightly build just right now :

  1. Hit return for a new line.
  2. Press "Insert Bulleted list" - Button
  3. Press "Insert Link" - Button
  4. enter www.blah.org as target and hit return to return back to gui
  5. Hit return to get new bullet.
  6. type "asd"

provides as source :

<p>This is some <strong>sample text</strong>. You are using <a href="http://www.fckeditor.net/">FCKeditor</a>.</p>
<ul>
    <li><a href="http://www.blah.org">www.blah.org</a></li>
    <li><a href="http://www.blah.org">asd</a></li>
</ul>
<p>&nbsp;</p>

Meaning, the <a>-Tag from the first bullet has been inherited by the second one, which is not what users expect, I guess.

Changed 15 months ago by fredck

  • keywords Confirmed Firefox added
  • version set to FCKeditor 2.4.2
  • milestone set to FCKeditor 2.5

Confirmed with FF2. Ok with IE6.

Changed 15 months ago by fredck

Related to #393 (Maybe a DUP).

Changed 14 months ago by fredck

  • cc Pascal.KUSTNER@… added
  • keywords SD-COE added
  • description modified (diff)
  • summary changed from <a> whitn <li> to Hitting Enter after links continues the link in the new paragraph

It doesn't happen with lists only. Any kind of block with ends with a link will present the same problem.

For example:

  1. Load <p><a href="http://www.fckeditor.net/">FCKeditor</a></p> in the source.
  2. In WYSIWYG, click after "FCKeditor" and hit Enter.
  3. Type "Test".

You have this output:

<p><a href="http://www.fckeditor.net/">FCKeditor</a></p>
<p><a href="http://www.fckeditor.net/">Test</a></p>

While the expected is:

<p><a href="http://www.fckeditor.net/">FCKeditor</a></p>
<p>Test</p>

Changed 14 months ago by fredck

  • cc jean-marc.libs@…, patrice.weber@… added

Changed 14 months ago by fredck

  • cc pkdille, Jyhem, nyloth added; Pascal.KUSTNER@…, jean-marc.libs@…, patrice.weber@… removed

Changed 14 months ago by martinkou

  • owner set to martinkou
  • status changed from new to assigned

Changed 13 months ago by fredck

  • component changed from General to UI : Enter Key

Changed 13 months ago by martinkou

  • status changed from assigned to closed
  • resolution set to fixed

I found the fixes to #393 (i.e. the changesets [596], [597], [598], [599]) to have also fixed this ticket. I can no longer reproduce the bug in the current SVN build.

Changed 12 months ago by Jyhem

  • status changed from closed to reopened
  • resolution deleted

I just tested on http://www.fckeditor.net/nightly/fckeditor/_samples/default.html and the bug is still present.

What I did: 1/ hit return 2/ click on "bulleted list" 3/ click on "Insert/Edit Link", then enter "www.example.com" in the pop-up and click OK 4/ set cursor on end of line, hit "return" 5/ a second bullet appears, I type jhjfgfg and this new text has inherited the link to www.example.com :-(

This is on FF 1.5 I did not test on IE6 because today's demo URL just shows garbage on my IE6

Changed 12 months ago by martinkou

I'm quite sure that you're having old cached JavaScript code in your browser, because when I tested the nightly build page on Firefox 1.5, the whole editor didn't even appear. (I've just fixed this problem in [730], by the way.)

I then tested the same nightly build page with Firefox 2, with your procedure, and the bug did not appear. Testing build 730 on Firefox 1.5 also did not invoke the bug.

http://www.youtube.com/watch?v=SUtl3opGVqE

Maybe you should try to clear the local cache of your browser first, and reload the nightly test page and see if the bug is still there. r730 probably wouldn't appear on the nightly test page until late tonight, so if you completely reload the nightly page with Firefox 1.5 the editor wouldn't work. You can still test it with a Portable Firefox 2, however.

Changed 12 months ago by fredck

With FF2, I'm able to reproduce the problem, by following the precise steps described by Jyhem.

Changed 12 months ago by martinkou

Ok I see what's wrong now. The bug is triggered when the caret position is set to the end by the mouse. Previously I set the caret position to the end by the keyboard which didn't trigger the bug.

Changed 12 months ago by martinkou

  • status changed from reopened to closed
  • resolution set to fixed

Fixed with [742].

Click here for more info about our SVN system.

Changed 12 months ago by aleksey

  • status changed from closed to reopened
  • resolution deleted

This bug is not fixed for Opera 9.5 alpha (build 9500) and Opera 9.23

Changed 11 months ago by martinkou

Applying the same kludge in Gecko to Opera fixes the problem but results in another bug. With the kludge on, whenever the caret is at the end of an anchor link, the caret cannot be moved by the left arrow key. Also, an extra backspace would be needed to delete the last character in the anchor link.

Changed 11 months ago by martinkou

  • status changed from reopened to closed
  • resolution set to fixed

Fixed with [812] and [813].

Click here for more info about our SVN system.

Note: See TracTickets for help on using tickets.