Looks like we can't reproduce the issue or there's a problem in the test case provided.
  1. #1
    Ext JS Premium Member
    Join Date
    Apr 2010
    Location
    Omaha, NE
    Posts
    557
    Vote Rating
    25
    estesbubba will become famous soon enough estesbubba will become famous soon enough

      0  

    Default ActionSheet defines cls instead of baseCls

    ActionSheet defines cls instead of baseCls


    See

    http://docs.sencha.com/touch/2-0/sou...xt-ActionSheet

  2. #2
    Sencha User Jamie Avins's Avatar
    Join Date
    Mar 2007
    Location
    Redwood City, California
    Posts
    3,661
    Vote Rating
    20
    Jamie Avins is a jewel in the rough Jamie Avins is a jewel in the rough Jamie Avins is a jewel in the rough

      0  

    Default


    Thank you for the report.

    Sencha Inc

    Jamie Avins

    @jamieavins

  3. #3
    Sencha User Jamie Avins's Avatar
    Join Date
    Mar 2007
    Location
    Redwood City, California
    Posts
    3,661
    Vote Rating
    20
    Jamie Avins is a jewel in the rough Jamie Avins is a jewel in the rough Jamie Avins is a jewel in the rough

      0  

    Default


    This is intentional. It should get both the baseCls of sheet and action-sheet.

    Sencha Inc

    Jamie Avins

    @jamieavins

  4. #4
    Ext JS Premium Member
    Join Date
    Apr 2010
    Location
    Omaha, NE
    Posts
    557
    Vote Rating
    25
    estesbubba will become famous soon enough estesbubba will become famous soon enough

      0  

    Default


    Quote Originally Posted by Jamie Avins View Post
    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?

  5. #5
    Sencha User Jamie Avins's Avatar
    Join Date
    Mar 2007
    Location
    Redwood City, California
    Posts
    3,661
    Vote Rating
    20
    Jamie Avins is a jewel in the rough Jamie Avins is a jewel in the rough Jamie Avins is a jewel in the rough

      0  

    Default


    cls accepts arrays, you can add as many as you need.

    Sencha Inc

    Jamie Avins

    @jamieavins

  6. #6
    Ext JS Premium Member
    Join Date
    Apr 2010
    Location
    Omaha, NE
    Posts
    557
    Vote Rating
    25
    estesbubba will become famous soon enough estesbubba will become famous soon enough

      0  

    Default


    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:

    Code:
    Ext.create('Ext.ActionSheet', {
        cls: [Ext.baseCSSPrefix + 'sheet-action', 'my-custom-cls'],
        ...
    });

  7. #7
    Sencha User Jamie Avins's Avatar
    Join Date
    Mar 2007
    Location
    Redwood City, California
    Posts
    3,661
    Vote Rating
    20
    Jamie Avins is a jewel in the rough Jamie Avins is a jewel in the rough Jamie Avins is a jewel in the rough

      0  

    Default


    Fair enough. We'll change it over to a baseCls.

    Sencha Inc

    Jamie Avins

    @jamieavins

  8. #8
    Ext JS Premium Member
    Join Date
    Apr 2010
    Location
    Omaha, NE
    Posts
    557
    Vote Rating
    25
    estesbubba will become famous soon enough estesbubba will become famous soon enough

      0  

    Default


    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.

  9. #9
    Sencha User Jamie Avins's Avatar
    Join Date
    Mar 2007
    Location
    Redwood City, California
    Posts
    3,661
    Vote Rating
    20
    Jamie Avins is a jewel in the rough Jamie Avins is a jewel in the rough Jamie Avins is a jewel in the rough

      0  

    Default


    We appreciate all the feedback we can get. The more our developers can share with us, the better we can make Touch 2.

    Sencha Inc

    Jamie Avins

    @jamieavins

Thread Participants: 1