Results 1 to 3 of 3

Thread: Ext.device.notification.PhoneGap does not return pressed button in lowercase

    Success! Looks like we've fixed this one. According to our records the fix was applied for TOUCH-2885 in a recent build.
  1. #1
    Sencha User
    Join Date
    Mar 2012
    Location
    Leeuwarden, Netherlands
    Posts
    20
    Vote Rating
    0
      0  

    Default Ext.device.notification.PhoneGap does not return pressed button in lowercase

    The callback for Ext.device.notification.PhoneGap receives the button value exactly as it was specified under buttons, whereas Ext.device.notifcation.Simulator en Ext.device.notification.Sencha return the value in lowerCase.

    Proposed fix for Ext.device.notification.PhoneGap on line 13:

    Code:
    config.callback.apply(config.scope, (config.buttons) ? [config.buttons[index - 1]].toLowerCase() : []);

  2. #2
    Sencha - Sr Software Engineer mitchellsimoens's Avatar
    Join Date
    Mar 2007
    Location
    Gainesville, FL
    Posts
    39,551
    Vote Rating
    1271
      0  

    Default

    Thanks for the report.
    Mitchell Simoens @LikelyMitch
    Sencha Inc, Senior Software Engineer
    ________________
    Learn BBCode and use it! Checkout the CODE tag!

    Check out my GitHub, lots of nice things for Ext JS and Sencha Touch
    https://github.com/mitchellsimoens

  3. #3
    Sencha User rdougan's Avatar
    Join Date
    Oct 2008
    Posts
    1,159
    Vote Rating
    7
      0  

    Default

    Fixed for the next release. Thanks!
    Sencha Inc.
    Robert Dougan - @rdougan
    Sencha Touch 2 and Ext JS 4 Core Team Member, SASS/Theming Wizard.

Tags for this Thread

Posting Permissions

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