Opened 15 years ago
Closed 15 years ago
#4859 closed Bug (fixed)
Flash object elements are broken in FF
Reported by: | Wiktor Walc | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.1 |
Component: | General | Version: | 3.0 |
Keywords: | Confirmed Review+ | Cc: |
Description (last modified by )
Confirmed on FF 3.5.5 @ WinXP/SP3.
Steps to reproduce
- start with
<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 source mode
- result:
<p><object height="525" width="660"> <param name="movie" value="http://www.youtube.com/v/HwNXvQ2OK_k&hl=pl_PL&fs=1&color1=0x234900&color2=0x4e9e00&border=1" /> </object> <param name="allowFullScreen" value="true" /> <param name="allowscriptaccess" value="always" /> <embed allowfullscreen="true" allowscriptaccess="always" height="525" src="http://www.youtube.com/v/HwNXvQ2OK_k&hl=pl_PL&fs=1&color1=0x234900&color2=0x4e9e00&border=1" type="application/x-shockwave-flash" width="660"> </embed></p>
(<embed>
and some<param>
elements are outside<object>
element)
Attachments (1)
Change History (6)
comment:1 Changed 15 years ago by
Description: | modified (diff) |
---|
comment:2 Changed 15 years ago by
Description: | modified (diff) |
---|---|
Milestone: | CKEditor 3.1 → CKEditor 3.3 |
Defer it, at least it works for valid HTML.
Changed 15 years ago by
Attachment: | 4859.patch added |
---|
comment:3 Changed 15 years ago by
Keywords: | Review? added |
---|---|
Milestone: | CKEditor 3.3 → CKEditor 3.1 |
Owner: | set to Garry Yao |
Status: | new → assigned |
The fixing seems to be simple, so let's have it back in 3.1...
Including the latest patch from #4856 for easy reviewing.
comment:4 Changed 15 years ago by
Keywords: | Review+ added; Review? removed |
---|
Please do not mix fixes from other tickets.
This does not happen when params are created this way:
it only happens when params are created like this:
(YouTube does like that).