PDA

View Full Version : [CLOSED] BUG & FIX: Adding panels to carousel shows too many circles in the indicator



Jonny Roller
8 Nov 2011, 8:43 AM
REQUIRED INFORMATION

Ext version tested:

Sencha Touch 2.0
Browser versions tested against:

Chrome 15
DOCTYPE tested against:

html
Description:

Dynamically adding panels to a carousel shows too many circles within the indicator
Steps to reproduce the problem:

See code below
The result that was expected:

Two circles within the indicator
The result that occurs instead:

Four circles within the indicator
Test Case:
// BUG: When done, four dots will appear in the indicator
// FIX: Carousel/onRemove should do a removeIndicator(), not an addIndicator()


Ext.application({
name: 'test',
launch: function() {


Ext.create('Ext.TabPanel', {
fullscreen: true,
tabBarPosition: 'bottom',


items: [
{
title: 'Home',
iconCls: 'home',


listeners: {
activate: function() {
this.removeAt(0);
this.add(new Ext.Panel({ html: 'First new panel' }));
this.add(new Ext.Panel({ html: 'Second new panel' }));
}
},


xtype: 'carousel',
items: [
{
html: 'Existing panel'
}
]
}
]
});
}
});


HELPFUL INFORMATION

Screenshot or Video:

None
See this URL for live test case:
(http://)
Debugging already done:

See fix below
Possible fix:

In Carousel.onRemove(), change addIndicator() to removeIndicator()
Additional CSS used:

only default
Operating System:

Windows 7

Jamie Avins
8 Nov 2011, 11:03 AM
Thank you for the report, I noticed the same thing and it has already been fixed.