Looks like we can't reproduce the issue or there's a problem in the test case provided.
ActionSheet defines cls instead of baseCls
Thank you for the report.
This is intentional. It should get both the baseCls of sheet and action-sheet.
I confused on if this was a bug and fixed or intentional? If ActionSheet does use cls then how can we add our own CSS to the ActionSheet?
Originally Posted by Jamie Avins
cls accepts arrays, you can add as many as you need.
Jamie - I still think ActionClass should use baseCls instead of cls just like all the other Ext components do. Sure, cls takes an array but then I kill the base class for ActionClass which isn't desirable. If ActionSheet needs also needs Sheet's CSS you could do:
baseCls: Ext.baseCSSPrefix + 'sheet ' + Ext.baseCSSPrefix + 'sheet-action'
That would allow the users to still add their own CSS to ActionSheet.
Otherwise we are left with doing this which doesn't seem clean and prone to error if the name changes:
cls: [Ext.baseCSSPrefix + 'sheet-action', 'my-custom-cls'],
Fair enough. We'll change it over to a baseCls.
Hope I'm not being a pain, but we have two Touch 2.0 development projects going on right now. Hopefully we're doing good testing of the framework and giving you guys useful feedback.
We appreciate all the feedback we can get. The more our developers can share with us, the better we can make Touch 2.