Opened 7 years ago

Closed 7 years ago

#5578 closed Bug (fixed)

Flash with Alignment is not shown properly in Editor body

Reported by: Satya Minnekanti Owned by: brooks
Priority: Normal Milestone: CKEditor 3.3
Component: General Version: 3.2
Keywords: IBM Confirmed Review+ Cc: Damian, joek

Description

To reproduce the defect:

  1. Open Ajax sample.
  1. Type some text and insert flash with right alignment.
  1. See that flash is inserted in the body of the editor and is shown as aligned to right.
  1. Type some more text and Click on Remove Editor button to save the page.
  1. See that Flash is shown with proper Alignment on Page View.
  1. Click on Create Editor button to open the page and see the Alignment of Flash in the boy of the Editor.

Expected Result:

Flash will be shown on the right hand side of Editor since it's Alignment was set to Right

Actual Result:

Flash is shown with out any Alignment and it is shown in the middle of the text when i check the Flash Alignment in Flash Properties it is showing Alignment correctly as Right

Attachments (3)

5578.patch (579 bytes) - added by brooks 7 years ago.
5578_2.patch (601 bytes) - added by brooks 7 years ago.
5578_3.patch (555 bytes) - added by brooks 7 years ago.

Download all attachments as: .zip

Change History (14)

comment:1 Changed 7 years ago by brooks

Keywords: Confirmed added
Owner: set to brooks
Status: newassigned

Changed 7 years ago by brooks

Attachment: 5578.patch added

comment:2 Changed 7 years ago by brooks

Keywords: review? added

comment:4 Changed 7 years ago by Garry Yao

Keywords: review+ added; review? removed

Ye, let's do that for safe.

comment:5 Changed 7 years ago by brooks

Resolution: fixed
Status: assignedclosed

fixed with 5414 5414

comment:6 Changed 7 years ago by Alfonso Martínez de Lizarrondo

Keywords: Review- added; review+ removed
Resolution: fixed
Status: closedreopened

Due to that checkin now it isn't possible to create a Page Break. In Firefox there's no error, but nothing happens, and in IE8 it throws an error pointing exactly at that line.

Changed 7 years ago by brooks

Attachment: 5578_2.patch added

comment:7 Changed 7 years ago by brooks

Keywords: Review? added; Review- removed

yes,add align property in
http://dev.fckeditor.net/browser/CKEditor/trunk/_source/plugins/fakeobjects/plugin.js#L67[[BR]] will make page break down.recover it now

comment:8 Changed 7 years ago by Garry Yao

Keywords: Review- added; Review? removed

We benefit with the change that fake element will automatically present alignment of the real element, I believe the problem is with the line itself:

align : realElement.attributes.align || ''
V.S.
align : realElement.getAttribute( 'align' )|| ''

Changed 7 years ago by brooks

Attachment: 5578_3.patch added

comment:9 Changed 7 years ago by brooks

Keywords: Review? added; Review- removed

nice suggest:-)

comment:10 Changed 7 years ago by Garry Yao

Keywords: Review+ added; Review? removed

There's no changelog needed.

comment:11 Changed 7 years ago by brooks

Resolution: fixed
Status: reopenedclosed

fixed with 5426 5426

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