FF: Dialogs shrink in width in V2 skin when dragged to the right of the screen

[James Cunningham]

Steps to reproduce the defect:

1. Open the skins sample in Firefox.

2. Open the Table Properties dialog in the V2 skin.

3. Drag the dialog to the right of the screen (as far right as possible).

Result: The dialog shrinks in width (see screenshot). This happens for all dialogs when dragged to the right of the screen. It also happens in the office skin.

The problem was introduced in 3.5 in changeset ​6118 when the getSize function in the dialog/plugin.js was changed from

getSize : function()
{
	if ( !this._.updateSize )
		return this._.size;
	var element = this._.element.getFirst();
	var size = this._.size = { width : element.$.offsetWidth || 0, height : element.$.offsetHeight || 0};

	// If either the offsetWidth or offsetHeight is 0, the element isn't visible.
	this._.updateSize = !size.width || !size.height;

	return size;
}

to

getSize : function()
{
	var element = this._.element.getFirst();
	return { width : element.$.offsetWidth || 0, height : element.$.offsetHeight || 0};
}

The problem can be resolved by using the old getSize function.

[Jakub Ś]

Dialogs get reduced in FF (3.6, 4) browsers.

jamescun - I'm not sure if the revision number you have given is correct. I have tested this revision and everything seems to work there. I think that revision number for which problems have started is [6241] but of course I may be wrong.

[James Cunningham]

Replying to j.swiderski:

Dialogs get reduced in FF (3.6, 4) browsers.

jamescun - I'm not sure if the revision number you have given is correct. I have tested this revision and everything seems to work there. I think that revision number for which problems have started is [6241] but of course I may be wrong.

I did not check to see which revision this defect was introduced so you are probably right. I got the changeset ​6188 from the changes that were delivered for ticket ​#5084

[Piotrek Koszuliński]

I could not reproduce this any more. Checked moono and kama.