Opened 11 years ago

Closed 11 years ago

#2965 closed Bug (fixed)

plugin:image preview area failed to update 'align' option

Reported by: Garry Yao Owned by: Artur Formella
Priority: Normal Milestone: CKEditor 3.0
Component: General Version: SVN (FCKeditor) - Retired
Keywords: Confirmed Review+ Cc:

Description

Procedures

  1. Open the replace by code example page in IE;
  2. Click image button to open dialog, input
    http://dev.fckeditor.net/chrome/site/logos.gif
    
    as image url.
  3. Change the Align select to option 'right', check the preview area.
  • Actual Result : Preview area remains the same.

Attachments (3)

2965.patch (1.1 KB) - added by Artur Formella 11 years ago.
2965_2.patch (1.4 KB) - added by Artur Formella 11 years ago.
2965_3.patch (1.7 KB) - added by Artur Formella 11 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 Changed 11 years ago by Artur Formella

Owner: set to Artur Formella
Status: newassigned

Changed 11 years ago by Artur Formella

Attachment: 2965.patch added

comment:2 Changed 11 years ago by Artur Formella

Keywords: Review? added

comment:3 Changed 11 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed

The patch worked well with Firefox, but has no effect in IE.

Changed 11 years ago by Artur Formella

Attachment: 2965_2.patch added

comment:4 Changed 11 years ago by Artur Formella

Keywords: Review? added; Review- removed

comment:5 Changed 11 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed

Please set this.getValue() to a local variable at the top of the commit function and reuse it inside the function body.

Changed 11 years ago by Artur Formella

Attachment: 2965_3.patch added

comment:6 Changed 11 years ago by Artur Formella

Keywords: Review? added; Review- removed

comment:7 Changed 11 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:8 Changed 11 years ago by Artur Formella

Resolution: fixed
Status: assignedclosed

Fixed with [3262]

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