SplitContainer: only resizes in FF 2 or 3 times before freezing

[jpsykes]

SlideContainer? widget in an implementation was being observed that it would resize 2-5 times and then just freeze. the bar was still clickable, but you couldn't drag it up or down, it just froze.

[jpsykes]

The issue is that in dijit.layout.SplitContainer?.beginSizing there are two tests for a variable, the test is looking for undefined, but rather than looking for typeof undefined, it just tests for foo. The problem occurs in that sometimes after a resize that value might be zero, zero is an accepted value, but gets interpreted as undefined, and the script falls to pieces. It results in a whole series of NaN's that prevent the slider from functioning again. A fix was found to be just changing:

--- Original Dojo 1.2.0 +++ Fix for split container freeze bug @@ -24,18 +24,16 @@

TODO: REVISIT - we want MARGIN_BOX and core hasn't exposed that yet (but can't we use it anyway if we pay attention? we do elsewhere.) this.originPos = dojo.coords(children[0].domNode, true); if(this.isHorizontal){

+ var client = (typeof e.layerX === 'undefined') ? e.offsetX : e.layerX;

• var client = (e.layerX ? e.layerX : e.offsetX);

var screen = e.pageX; this.originPos = this.originPos.x;

}else{

+ var client = (typeof e.layerY === 'undefined') ? e.offsetY : e.layerY;

• var client = (e.layerY ? e.layerY : e.offsetY);

var screen = e.pageY; this.originPos = this.originPos.y;

} this.startPoint = this.lastPoint = screen; this.screenToClientOffset = screen - client; this.dragOffset = this.lastPoint - this.paneBefore.sizeActual - this.originPos - this.paneBefore.position;

if(!this.activeSizing){

I know this is old and deprecated code, but I wanted to post it here incase someone else has a similar issue and came here to see if there was a solution.

[jpsykes]

SplitContainer? Fix Diff file

[jpsykes]

Garbled the code in the above comment, so I've attached the diff.

[jpsykes]

Further info. It's fixed in the 1.4.3 version using this code.

if(this.isHorizontal){

var client = e.layerX

e.offsetX

0;

var screen = e.pageX; this.originPos = this.originPos.x;

}else{

var client = e.layerY

e.offsetY

0;

var screen = e.pageY; this.originPos = this.originPos.y;

}

[bill]

Dup of #7424.