TabContainer: closing focused tab in IE 7 cause error

[dvdwalt]

Hi,

I'm using the sample code from Dojo Campus in the attached file, and added the closable: true attribute to the second and third tabs.

The code does the following:

• Create one or more closable tabs through JavaScript code.

The problem in IE 7 is the following:

• When a closable tag does not have focus and it is closed, it closes OK.
• When a closable tab has focus and is closed in IE 7 I get an error message to the like of 'null' is null or not an object.
• The tab is closed, but the focus is not turned to any tab in the container, one gets sort of a blank pane.
• I suspect the other IE's will also have this problem.
• As usual Firefox works fine :)!

[dvdwalt]

Focused Closable Tab Error - Test in IE 7

[bill]

Thanks for the detailed description / test case.

Looks like a regression starting with the scrolling tab container rewrite, although the actual bug is in the pre-rewrite old code. StackContainer?.removeChild() calls layout() before a new tab has been selected, which calls tablist.resize() --> _getScrollForSelectedTab(), which references the recently destroyed this._selectedTab.

[bill]

(In [20707]) Clear this._selectedTab when the selected tab is deleted, in order to avoid referencing a deleted dom node. Fixes #10246 !strict.