Ticket #1383 (closed Bug: fixed)

Opened 5 years ago

Last modified 4 years ago

Hyperlink merges with text on <br /> backspace

Reported by: Scott Owned by: fredck
Priority: Normal Milestone: FCKeditor 2.6.1
Component: UI : Enter Key Version: SVN (FCKeditor) - Retired
Keywords: Confirmed IE Review+ Cc:

Description

To replicate:

<p><a href=" http://google.com/"> http://google.com</a><br /> Press backspace at the start of this line</p>

The link should only be auto-created when a space or an enter is placed at the end of a link.

Attachments

1383.patch Download (2.9 KB) - added by martinkou 4 years ago.
1383_2.patch Download (1.7 KB) - added by fredck 4 years ago.

Change History

comment:1 Changed 5 years ago by fredck

  • Milestone changed from FCKeditor 2.5 to FCKeditor 2.6

Here again we have IE and its automations. I'm targeting it to 2.6, to check if there is any chance to have it fixed.

comment:2 Changed 4 years ago by w.olchawa

  • Keywords Confirmed added

comment:3 Changed 4 years ago by fredck

  • Milestone changed from FCKeditor 2.6 to FCKeditor 2.6.1

Postponing it a little.

comment:4 Changed 4 years ago by martinkou

  • Owner set to martinkou
  • Status changed from new to assigned

Changed 4 years ago by martinkou

comment:5 Changed 4 years ago by martinkou

  • Keywords Review? added

comment:6 Changed 4 years ago by fredck

  • Keywords IE added

comment:7 Changed 4 years ago by fredck

  • Owner changed from martinkou to fredck
  • Status changed from assigned to new

The proposed patch may be ok. The idea is correct, but I think a simpler logic can be found. I'll attach a new patch for it.

Changed 4 years ago by fredck

comment:8 Changed 4 years ago by martinkou

  • Keywords Review+ added; Review? removed

comment:9 Changed 4 years ago by fredck

  • Status changed from new to closed
  • Resolution set to fixed

Fixed with [1996]. Click here for more info about our SVN system.

Note: See TracTickets for help on using tickets.
© 2003 – 2011 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy