1. #1
    Ext JS Premium Member stever's Avatar
    Join Date
    Mar 2007
    Posts
    1,407
    Vote Rating
    6
    stever will become famous soon enough stever will become famous soon enough

      0  

    Default [2.0b1] TabPanel with items with itemId instead of id

    [2.0b1] TabPanel with items with itemId instead of id


    In a TabPanel, where one of the tabs has itemId set (and no id set), the TabPanel incorrectly tries to recreate the id by which to get the component when you try and click on it:

    Code:
     findTargets : function(e){
    	var item = null;
    	var itemEl = e.getTarget('li', this.strip);
    	if(itemEl){
    		item = this.getComponent(itemEl.id.split('__')[1]);
    		if(item.disabled){
    On the last line, "item" is undefined. The id for this element needs to use something like ... (item.itemId||item.id) + '__' ... in its creation.

  2. #2
    Sencha - Community Support Team jsakalos's Avatar
    Join Date
    Apr 2007
    Location
    Slovakia
    Posts
    27,537
    Vote Rating
    380
    jsakalos has a brilliant future jsakalos has a brilliant future jsakalos has a brilliant future jsakalos has a brilliant future jsakalos has a brilliant future jsakalos has a brilliant future jsakalos has a brilliant future jsakalos has a brilliant future jsakalos has a brilliant future jsakalos has a brilliant future jsakalos has a brilliant future

      0  

    Default


    Thank you for posting. We will investigate it and it will be fixed.
    Jozef Sakalos, aka Saki

    Education, extensions and services for developers at new http://extjs.eu
    News: Grid Search Plugin, ExtJS 5 Complex Data Binding using MVVM


  3. #3
    Sencha User jack.slocum's Avatar
    Join Date
    Mar 2007
    Location
    Tampa, FL
    Posts
    6,955
    Vote Rating
    17
    jack.slocum will become famous soon enough jack.slocum will become famous soon enough

      0  

    Default


    I will be submitting some changes to SVN shortly.

    This is actually very problematic and may need to be removed. I am sure there are other spots in the codebase which could be affected.
    Jack Slocum
    Ext JS Founder
    Original author of Ext JS 1, 2 & 3.
    Twitter: @jackslocum
    jack@extjs.com

Thread Participants: 2