Opened 10 years ago

Closed 10 years ago

#4789 closed Bug (fixed)

[Safari] Changing image properties produces lots of duplicated styles

Reported by: Wiktor Walc Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.1
Component: UI : Dialogs Version: SVN (CKEditor) - OLD
Keywords: Confirmed Safari 3.0.2 Review+ Cc:

Description

Steps to reproduce:

  • paste the following code in source mode:
    <p>
    	<img align="left" alt="" height="168" hspace="10" src="http://a.cksource.com/c/1/inc/img/demo-little-red.jpg" width="120" /></p>
    
  • switch to wysiwyg mode
  • click on the image and open the image dialog
  • change border, hspace, vspace
  • click ok
  • open image dialog again
  • change border, hspace, vspace to some other values
  • click ok
  • result:
    <p>
    	<img alt="" src="http://a.cksource.com/c/1/inc/img/demo-little-red.jpg" style="border-color: initial; border-color: initial; border-color: initial; border-color: initial; border-top-width: 7px; border-right-width: 7px; border-bottom-width: 7px; border-left-width: 7px; border-top-style: solid; border-right-style: solid; border-bottom-style: solid; border-left-style: solid; border-color: initial; margin-left: 8px; margin-right: 8px; margin-top: 9px; margin-bottom: 9px; float: left; width: 120px; height: 168px; " /></p>
    
  • border-color: initial; is repeated a couple of times
  • the number of repeats increases every time we change border/hspace/vspace properties

This bug probably has been introduced in #4246.

Attachments (1)

4789.patch (1.8 KB) - added by Garry Yao 10 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 10 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.1

comment:2 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: 3.0.2 added

Changed 10 years ago by Garry Yao

Attachment: 4789.patch added

comment:3 Changed 10 years ago by Garry Yao

Component: GeneralUI : Dialogs
Keywords: Safari Review? added
Owner: set to Garry Yao
Status: newassigned

Regression of [4685].

comment:4 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:5 Changed 10 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [4725].

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