ST v2.1.1 - Ext.tab.Panel's delegateListeners won't fire!

[robert_k]

In our application we use the ability to disable a tab by disabling its panel quite frequently. I've been upgrading the same application from 2.0.3 to 2.1.1 and I've noticed that I can no longer disable tabs. In fact, when checking "getDisabled()" on the active panel it says "true" but the class wasn't applied and I can select the tab.

Not sure exactly where the bug resides, though I'm fairly sure it isn't with Ext.tab.Tab or with Ext.tab.Panel.

[robert_k]

I'm still not sure what's causing the error, but the problem is with "delegateListeners" for Ext.tab.Panel not being triggered. I can manually enable/disable a tab and it works properly, but if I enable/disable the panel the delegateListeners won't fire. The same happens for hidden.

I don't know that this issue is unique to Ext.tab.Panel, so this could be causing problems elsewhere (and probably is).

[mitchellsimoens]

Looks like it needs to be removed as it's no longer used at all. delegateListeners is marked as private so looks like it is going away.

[robert_k]

Okay, but that reinforces the problem: there's nothing handling hidden/disabled/docked/floating property changes and synchronizing them with the tab button. This is in the current 2.1.1 release version no less. I can work around this by enabling/disabling the panel.tab property.

If this were really "going away" it should either: work and show a deprecation notice, or be completely replaced.

[Jamie Avins]

This is indeed a mistake to still have the delegateListeners object in place. It has been removed and the listener has been put back in place in the initialize method:

    initialize: function() {
        this.callParent();

        this.on({
            order: 'before',
            activetabchange: 'doTabChange',
            delegate: '> tabbar',
            scope   : this
        });

        this.on({
            disabledchange: 'onItemDisabledChange',
            delegate: '> component',
            scope   : this
        });
    },