Opened 7 years ago

Closed 7 years ago

Last modified 6 years ago

#7300 closed Bug (fixed)

Link dialog does not load "Left/Top Position" values properly

Reported by: Wiktor Walc Owned by: Sa'ar Zac Elias
Priority: Normal Milestone: CKEditor 3.5.3
Component: General Version: 3.0
Keywords: Cc: rocco.tschirner@…

Description

Steps to reproduce

  • Insert in the link-dialog as target frame a popup-window and set "Left Position" and/or "Top Position" to "1"
  • Re-open the dialog to edit the link
  • The editor now shows "true" in both fields

Before:

<a href="http://example.com" onclick="window.open(this.href, '', 'resizable=no,status=no,location=no,toolbar=no,menubar=no,fullscreen=no,scrollbars=no,dependent=no,left=1,top=1'); return false;">http://example.com</a>

After:

<a href="http://example.com" onclick="window.open(this.href, '', 'resizable=no,status=no,location=no,toolbar=no,menubar=no,fullscreen=no,scrollbars=no,dependent=no,left=true,top=true'); return false;">http://example.com</a>

Attachments (1)

7300.patch (775 bytes) - added by Sa'ar Zac Elias 7 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 7 years ago by Wiktor Walc

Status: newconfirmed

Changed 7 years ago by Sa'ar Zac Elias

Attachment: 7300.patch added

comment:2 Changed 7 years ago by Sa'ar Zac Elias

Owner: set to Sa'ar Zac Elias
Status: confirmedreview

Same for "height" and "width" fields.

comment:3 Changed 7 years ago by Garry Yao

Status: reviewreview_passed

comment:4 Changed 7 years ago by Sa'ar Zac Elias

Resolution: fixed
Status: review_passedclosed

Fixed with [6547].

comment:5 Changed 6 years ago by Frederico Caldeira Knabben

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