Opened 8 years ago

Closed 8 years ago

#4856 closed Bug (fixed)

CKEditor leaving tags when removing flash objects

Reported by: Wiktor Walc Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.1
Component: General Version: 3.0
Keywords: Confirmed Review+ Cc:

Description

Steps to reproduce

  • paste the following in source mode:
    <p><object width="660" height="525"><param name="movie" value="http://www.youtube.com/v/HwNXvQ2OK_k&hl=pl_PL&fs=1&color1=0x234900&color2=0x4e9e00&border=1"></param><param name="allowFullScreen" value="true"></param><param name="allowscriptaccess" value="always"></param><embed src="http://www.youtube.com/v/HwNXvQ2OK_k&hl=pl_PL&fs=1&color1=0x234900&color2=0x4e9e00&border=1" type="application/x-shockwave-flash" allowscriptaccess="always" allowfullscreen="true" width="660" height="525"></embed></object></p>
    
  • switch to wysiwyg mode
  • press "Delete" button
  • result:
    <p>
    	<object height="525" width="660"><param name="movie" value="http://www.youtube.com/v/HwNXvQ2OK_k&amp;hl=pl_PL&amp;fs=1&amp;color1=0x234900&amp;color2=0x4e9e00&amp;border=1" /><param name="allowFullScreen" value="true" /><param name="allowscriptaccess" value="always" /></object></p>
    
  • expected result: the whole object should be removed (it was working in CKEditor 3.0 RC)

This is a regression bug introduced in [3779].

Attachments (1)

4856.patch (586 bytes) - added by Garry Yao 8 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 8 years ago by Garry Yao

Owner: set to Garry Yao
Status: newassigned

Changed 8 years ago by Garry Yao

Attachment: 4856.patch added

comment:2 Changed 8 years ago by Garry Yao

Keywords: Review? added

comment:3 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:4 Changed 8 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [4789].

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