Opened 14 years ago

Closed 14 years ago

#4993 closed Bug (fixed)

Cannot change image alignment back to "not set" without reopening a dialog

Reported by: Wiktor Walc Owned by: Minh Nguyen
Priority: Normal Milestone: CKEditor 3.2
Component: General Version: 3.0.2
Keywords: Confirmed Review+ Cc:

Description

When embedding an image, if you change the alignment dropdown to either left or right, you can't change it back to "<not set>" without reopening the dialog.

Steps to reproduce

  • Start with
    <h1>
    	<img alt="" src="http://a.cksource.com/c/1/inc/img/fireworks.png" />Happy New Year!</h1>
    <p>
    	The CKSource team wishes a <strong>wonderful and happy New Year</strong> to everybody around the globe!</p>
    
  • open image dialog, change alignment to right, change alignment to <not set>
  • result:
    <img alt="" src="http://a.cksource.com/c/1/inc/img/fireworks.png" style="float: right;" />
    

(float: right has not been removed).

This bug has been introduced in [4685] (#4246).

Attachments (1)

4993.patch (669 bytes) - added by Minh Nguyen 14 years ago.

Download all attachments as: .zip

Change History (5)

Changed 14 years ago by Minh Nguyen

Attachment: 4993.patch added

comment:1 Changed 14 years ago by Minh Nguyen

Keywords: Review? added
Owner: set to Minh Nguyen
Status: newassigned

comment:2 Changed 14 years ago by Minh Nguyen

In this case we don't need using method this.isChanged() in condition.

comment:3 Changed 14 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

That's true, we don't need to retain float values that are not supported.

comment:4 Changed 14 years ago by Minh Nguyen

Resolution: fixed
Status: assignedclosed

Fixed with [4994].

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy