IE: Height value missing from iFrame dialog

[Satya Minnekanti]

To reproduce the defect:

1. Open CK Editor and click on IFrame icon.

2. Enter vales for URL, width(100),height(100) and click OK button.

3. open Context menu on inserted Iframe and click on IFrame properties option.

Expected Result: IFrame Properties dialog comes up and shows correct values for URL,width & height that we entered in step 2.

Actual Result: IFrame Properties dialog comes up,shows correct values for URL,width that we entered in step 2 but Height value is shown empty.

[Jakub Ś]

I was able to reproduce it in IE6, IE7 and IE8. In IE9 it seems to be fine.

This issue has been occurring from rev [6958]

NOTE:If you enter height first and then width then the later one will be missing. So it is about both dimensions and not height only.

[Garry Yao]

The real problem behind is that, after [6958] iframe dialog use both the attribute and inline style to present dimension, which are conflicting with each other, but why haven't it been aligned with other style-based dialog (like image and table)? Well that's just because a historical reason of fake object which are more convenient to manipulate attributes than with styles.

The proposed patch eliminates attribute based dimension and introduces necessary changes to make this happen easier.

[Sa'ar Zac Elias]

The width/height attributes should be deprecated in the flash dialog as well at the same scope of changing the fake images' attributes based dimensions.

[Garry Yao]

I was trying to avoid a broader range of changes, but it's ok if you think we should get them aligned at one time.

[Sa'ar Zac Elias]

The ticket TC fails with the last patch.

[Sa'ar Zac Elias]

Latest patch fixes the original TC and contains minor improvements.

[Sa'ar Zac Elias]

Insert the following embed-less object:

<object width="251" height="200" data="http://releases.flowplayer.org/swf/flowplayer-3.2.5.swf" type="application/x-shockwave-flash">
	<param name="movie" value="http://releases.flowplayer.org/swf/flowplayer-3.2.5.swf" />
	<param name="allowfullscreen" value="true" />
	<param name="allowscriptaccess" value="always" />
</object> 

1. Switch back to source. An empty style attribute is being added.
2. Open the dialog with the context menu. The width and height attributes are not loaded.
3. Set some width/height and click OK. JS error is thrown.

[Sa'ar Zac Elias]

The latest patch fixes those problems, along with some code style fixes and KISS opportunities.

[Garry Yao]

Oops, back-compact is broken by my previous patch, but I'm afraid that force convertion wouldn't be an option, as it mangles user's content on data output, let's temporarily follow the current conversion in image dialog - attributes are dropped only if the dimension is modified by dialogs. I'll come with a new patch.

[Sa'ar Zac Elias]

• IE: Ticket TC is not resolved.
• Insert an iframe, leaving the width/height fields blank. Result: fake image is sized 1*1.
• With the contents in comment:9, open the dialog and go to the advanced tab. In the styles field, put "width:80px". Switch to general tab and see that the height field disappears.

[Garry Yao]

As more problems are found with the patch, I'd fallback the patch to only tackle the original issue of this ticket and conduct the continuation at #7958.

[Sa'ar Zac Elias]

The event should not be fired if the field hasn't changed.

[Garry Yao]

Fixed with [7001].