Success! Looks like we've fixed this one. According to our records the fix was applied for TOUCH-1872 in a recent build.
  1. #1
    Sencha Premium Member
    Join Date
    Aug 2010
    Posts
    27
    Vote Rating
    1
    jensen83 is on a distinguished road

      0  

    Default [PR4] broken override (Ext.LoadMask)

    [PR4] broken override (Ext.LoadMask)


    Override does not give expected behaviour... the next example should alert "testing", but instead it gives "Loading..." the config message defined in sencha.

    Is it just the case with LoadMask or a global bug?

    Code:
    Ext.define('Ext.override.LoadMask', {
      override: 'Ext.LoadMask',
      config: {
        message: 'testing', // FIX
        messageCls: Ext.baseCSSPrefix + 'mask-message',
        indicator: true
      }
    });
    
    alert((new Ext.LoadMask()).getMessage());

  2. #2
    Sencha User Jamie Avins's Avatar
    Join Date
    Mar 2007
    Location
    Redwood City, California
    Posts
    3,661
    Vote Rating
    18
    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 was done due to a problem with centering the Loadmask on scollable containers. This will return in the next release.

    Sencha Inc

    Jamie Avins

    @jamieavins

  3. #3
    Sencha Premium Member
    Join Date
    Aug 2010
    Posts
    27
    Vote Rating
    1
    jensen83 is on a distinguished road

      0  

    Default Problem still exists

    Problem still exists


    The test case still throws the wrong alert value...

  4. #4
    Sencha User Jamie Avins's Avatar
    Join Date
    Mar 2007
    Location
    Redwood City, California
    Posts
    3,661
    Vote Rating
    18
    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


    I can't follow this override logic you have in place. Why is this an override to an existing configuration or not just a class that extends it?

    Sencha Inc

    Jamie Avins

    @jamieavins

  5. #5
    Sencha Premium Member
    Join Date
    Aug 2010
    Posts
    27
    Vote Rating
    1
    jensen83 is on a distinguished road

      0  

    Default


    Ext.LoadMask is used in other classes. I don't want to extend all the other class, so I thought to override the base Ext.LoadMask.

    Is it not possible to override the config? How is the override meant to work?

Thread Participants: 1