Ticket #4724 (closed Bug: fixed)

Opened 4 years ago

Last modified 4 years 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 (954 bytes) - added by alfonsoml 4 years ago.
Proposed patch

Change History

comment:1 Changed 4 years ago by SebDIP

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

comment:2 Changed 4 years ago by SebDIP

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

comment:3 Changed 4 years 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 4 years ago by alfonsoml

Proposed patch

comment:4 Changed 4 years ago by alfonsoml

  • Owner set to alfonsoml
  • Status changed from new to assigned
  • Keywords Review? added

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

comment:5 Changed 4 years ago by fredck

  • Keywords Review+ added; Review? removed

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

comment:6 Changed 4 years ago by alfonsoml

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

Fixed with [4917]

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