Results 1 to 4 of 4

Thread: Animations z-index seems wrong

    This duplicates another bug already reported in our system: TOUCH-1170
  1. #1
    Sencha User
    Join Date
    Sep 2011
    Posts
    4
    Vote Rating
    0
      0  

    Default Animations z-index seems wrong

    This pattern seems to exist in 1.x and 2.x for animations:
    curZ = el.getStyle('z-index') == 'auto' ? 0 : el.getStyle('z-index'),

    ... later on... some math on curZ
    zIndex = curZ + 1

    If I'm using z-index to achieve some interesting effects then this makes it very hard to work with. If I set z-index: 100, then rather than the animation setting it to 101 (which seems reasonable), it get's set to 1001.

    Perhaps the pattern to set curZ in all the animations should be:
    curZ = el.getStyle('z-index') == 'auto' ? 0 : parseInt(el.getStyle('z-index'))

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

    Default

    Can I get a test case to reproduce?
    Mitchell Simoens @LikelyMitch
    Sencha Inc, Senior Software Engineer
    ________________
    Learn BBCode and use it!

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

    Think my support is good? Get more personalized support via a support subscription. https://www.sencha.com/store/

    Need more help with your app? Hire Sencha Services services@sencha.com

    Want to learn Sencha Touch 2? Check out Sencha Touch in Action that is in print!

    When posting code, please use BBCode's CODE tags.

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

    Default

    Animations are still being worked on, but I am going to add this as a bug to ensure it is looked at.
    Sencha Inc.
    Robert Dougan - @rdougan
    Sencha Touch 2 and Ext JS 4 Core Team Member, SASS/Theming Wizard.

  4. #4
    Sencha User
    Join Date
    Feb 2012
    Location
    Illinois
    Posts
    97
    Vote Rating
    2
      0  

    Default

    The problem is around line 379 of http://docs.sencha.com/touch/2.4/2.4....html#Ext-Anim: 'zIndex = currentZIndex + 1'

    If zIndex is set as el.getStyle('z-index') it is set as the string '1' and string math occurs causing the z-index to grow from '1' to '11' to '111'. If operating on the same component it only takes 7 animations to reach the max allowed z-index (2147483647)...2^31.

    I attached a fiddle that replicates the problem...if someone can figure out how to embed it that would be a welcome edit.
    Attached Files Attached Files

Posting Permissions

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