Results 1 to 2 of 2

Thread: Ext.MessageBox has both baseCls and cls config

    Success! Looks like we've fixed this one. According to our records the fix was applied for TOUCH-1387 in Sprint 14.
  1. #1
    Ext JS Premium Member
    Join Date
    Apr 2010
    Location
    Omaha, NE
    Posts
    631
    Vote Rating
    41
      0  

    Default Ext.MessageBox has both baseCls and cls config

    It seems that the standard is for a component to define baseCls and cls is used to add additional style to that instance. Ext.MessageBox defines both and the only I could find to add my CSS was to do this. It works, but probably not preferred:

    Code:
            msg = Ext.create('Ext.MessageBox', {
                title: 'Receipt',
                buttons: Ext.MessageBox.OK,
                scrollable: 'vertical',
                cls: Ext.baseCSSPrefix + 'panel' + ' c-print-receipt',
                items: items
            });
            msg.show();
    If Ext.MessageBox needs to define both baseCls and cls, can we get an extraCls config?

  2. #2
    Sencha User mitchellsimoens's Avatar
    Join Date
    Mar 2007
    Location
    Gainesville, FL
    Posts
    40,050
    Vote Rating
    1381
      0  

    Default

    There shouldn't be a reason why the framework shouldn't use baseCls and app code shouldn't use cls so we will take a look into this.
    Mitchell Simoens @LikelyMitch
    Modus Create, Senior Frontend Engineer
    ________________
    Need any sort of Ext JS help? Modus Create is here to help!

    Check out my GitHub:
    https://github.com/mitchellsimoens

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •