PDA

View Full Version : [FIXED-409] 0.96: Carousel Switch Broken when List is Used



don_bm
7 Oct 2010, 11:04 AM
If there is a list as a card in a carousel, switching cards is impossible. Here's a code sample that worked fine in 0.93 (I didn't try it on 0.94 or 0.95):



Ext.regModel('Story', {
fields: ['title']
});
var store = new Ext.data.Store({
model: 'Story',
data: [{
title: 'Story 1'
},{
title: 'Story 2'
},{
title: 'Story 3'
}]
});
var list = new Ext.List({
tpl: '<tpl for="."><div class="story"><strong>{title}</strong></div></tpl>',
store: store,
itemSelector: 'div.story',
scroll: false
});

var simpleCarousel = new Ext.Carousel({
items: [
list,
{
html: '<h1>Carousel</h1>',
cls: 'card1'
}, {
title: 'Tab 2',
html: '2',
cls: 'card2'
}, {
title: 'Tab 3',
html: '3',
cls: 'card3'
}]
});

var mainPanel = new Ext.Panel({
layout: 'fit',
flex: 1,
style: 'border: 5px solid #666666;background-color:#FFFFFF',
items: [simpleCarousel]
});

Ext.setup({
onReady: function() {
var homeScreen = new Ext.Panel({
layout: {
type: 'hbox',
align: 'stretch',
pack: 'start'
},
fullscreen: true,
items: [mainPanel]
});
}
});
Note that scroll is set to false for the list so this is a different bug than #377

Edit: Switching works but only after you first scroll vertically. So instead of just swiping to the left or right you need to swipe up or down first, basically making an 'L' type motion.

Another Edit: Switching using an 'L' type motion doesn't work if vertical scrolling is enabled.

BrendanC
7 Oct 2010, 11:18 AM
Thanks for the bug report.

alikic
8 Oct 2010, 4:55 AM
Same problem here, except my list (in the carousel card) is scrollable. Switching cards by swiping left or right doesn't work anymore. It used to work in 0.95. The only way to switch to another card is to tap on the bottom of the screen (left or right).

Generally, 0.96 is much less stable than 0.95 when it comes to handling gestures, probably due to the complete rework that you guys did (surprise, surprise;). For example, sorting lines (by dragging a handle) in a simple sortable (and scrollable) list, which used to work fine for me in 0.95 is screwed up in 0.96. Instead of just moving one (selected) element, now the whole list moves with the selected element, but at the different speed.

These, and few other glaring bugs introduced in 0.96, tell me that you guys are missing important test cases in your regression testing suite (you have one, right?) and rely on the community to do beta testing and report issues. Fair enough, I am not paying for the product yet, but I am hoping the regression will work better after the final release (when I buy the product).

don_bm
8 Oct 2010, 7:01 AM
These, and few other glaring bugs introduced in 0.96, tell me that you guys are missing important test cases in your regression testing suite (you have one, right?) and rely on the community to do beta testing and report issues. Fair enough, I am not paying for the product yet, but I am hoping the regression will work better after the final release (when I buy the product).


Totally agree with you. Upgrading to one incremental release is always an adventure. But I went from 0.93 to 0.96 which was 3 painful adventures rolled up in one...