Opened 7 years ago

Closed 6 years ago

#4901 closed Bug (fixed)

Can not modify target window properties of a link

Reported by: damo Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.1
Component: UI : Dialogs Version:
Keywords: Confirmed IBM Review+ Cc: JoeK

Description

When the target option of a link is Popup Window, the values for the Popup Window Features and sizes can not be modified after the first edit of the properties.

To reproduce:

  1. Create a new link
  2. Edit the target as Popup Window
  3. Select some Popup Window Features
  4. Set a width and height
  5. Click OK
  6. Remove editor & create editor
  7. Edit the link
  8. Modify the original edits to target, including width and height
  9. Click OK
  10. Edit the link again to confirm options at step 8. are kept.

Result: The options are not modified in step 8. but remain as set in steps 3 & 4

Attachments (1)

4901.patch (581 bytes) - added by garry.yao 6 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 7 years ago by damo

  • Summary changed from [FF] Can not modify target window properties of a link to Can not modify target window properties of a link

In FF, Popup Window Features, width, height, left position, right position and Popup Window Name do not get updated.

In IE, only Popup Window Name does not get updated.

comment:2 Changed 6 years ago by garry.yao

  • Component changed from General to UI : Dialogs
  • Keywords Confirmed added
  • Owner set to garry.yao
  • Status changed from new to assigned

Ticket Test added at :
http://ckeditor.t/tt/4901/1.html.

Changed 6 years ago by garry.yao

comment:3 Changed 6 years ago by garry.yao

  • Keywords Review? added

comment:4 Changed 6 years ago by fredck

  • Keywords Review+ added; Review? removed

comment:5 Changed 6 years ago by garry.yao

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

Fixed with [4821].

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