Ticket #4724 (closed Bug: fixed)

Opened 10 months ago

Last modified 8 months ago

[IE8] Create link hypertext

Reported by: SebDIP Owned by: alfonsoml
Priority: Normal Milestone: CKEditor 3.2
Component: General Version: SVN (CKEditor) - OLD
Keywords: Confirmed IE Review+ Cc:

Description

(sorry for my english level)

Using IE8 ,in the window for create link, if i paste " http://www.google.com" with right click mouse, and i valid with mouse. The link src contain two " http://"

Attachments

4724.patch Download (0.9 KB) - added by alfonsoml 8 months ago.
Proposed patch

Change History

Changed 10 months ago by SebDIP

  • summary changed from Crate link hypertext and IE8 to [IE8] Crate link hypertext

Changed 10 months ago by SebDIP

  • summary changed from [IE8] Crate link hypertext to [IE8] Create link hypertext

Changed 9 months ago by fredck

  • keywords Confirmed IE added; ie8 link right click paste removed
  • version set to SVN (CKEditor)
  • milestone changed from CKEditor 3.1 to CKEditor 3.2

Confirmed with IE8. Ok with FF.

Steps to reproduce:

  1. Copy the text " http://www.google.com/" to the clipboard.
  2. Select some text in the editor.
  3. Open the Link dialog.
  4. Right click the URL field and select "Paste".
  5. Confirm the dialog, closing it.
  6. Check the source.

Changed 8 months ago by alfonsoml

Proposed patch

Changed 8 months ago by alfonsoml

  • keywords Review? added
  • owner set to alfonsoml
  • status changed from new to assigned

I've included just a call to onChange() in the commit method of the control to be sure that it is updated properly.

This is a simple patch, another solution could be to modify the commit method of the CKEDITOR.ui.dialog.textInput prototype so all text inputs have this change, but I don't know if it's worth the effort

Changed 8 months ago by fredck

  • keywords Review+ added; Review? removed

Looks like a good and simple way to solve the problem.

Changed 8 months ago by alfonsoml

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

Fixed with [4917]

Note: See TracTickets for help on using tickets.