Hybrid View

    Thank you for reporting this bug. We will make it our priority to review this report.
  1. #1
    Ext JS Premium Member
    Join Date
    May 2007
    Posts
    13
    Vote Rating
    0
    cdonnellytx is on a distinguished road

      0  

    Default [OPEN-602] webkitTransitionEnd event does not always fire

    [OPEN-602] webkitTransitionEnd event does not always fire


    Sencha Touch version tested:
    • 1.0



    Platform tested against:
    • iOS 4
    • Android 2.1
    • Android 2.2
    • Google Chrome 7.0

    Description:
    Test Case:
    N/A. Basically, any transition (such as switching cards) should show the problem.

    If you are unable to replicate it, please let me know and I will set up a test case.


    Possible fix:
    Patch provided below. NOTE: the fix also includes a null check in onTransitionEnd itself, which was another issue we were seeing.
    Code:
    --- Anim-1.0.js	2010-11-15 17:25:26.000000000 -0600
    +++ Anim-Va.js	2010-11-23 15:10:07.310063900 -0600
    @@ -199,7 +199,27 @@
                     after: after
                 });
     
    -            for (property in config.to) {
    +            /**
    +             * cdonnelly/dhawkins 2010-09-02:
    +             * Because webkitTransitionEnd does not always seem to fire, we added a timeout 50ms longer than the actual event 
    +             * that checks to see if the event was fired, and if not, it completes the work the event should have done.
    +             * Should WebKit be fixed in the future, the setTimeout() can be removed 
    +             * (or at least not invoked if there is a way to detect the "good" vs. "bad" WebKit version(s)).  
    +             * 
    +             * @see http://www.cuppadev.co.uk/quirks/the-trouble-with-css-transitions/
    +             */
    +            setTimeout(function () {
    +                if (me.running[el.id]) {
    +                    me.onTransitionEnd(null, el, {
    +                        single: true,
    +                        config: config,
    +                        after: after
    +                    });
    +                    
    +                }
    +            }, config.duration + 50);
    +
    +            for (var property in config.to) {
                     if (!config.to.hasOwnProperty(property)) {
                         continue;
                     }
    @@ -213,7 +233,7 @@
     
         onTransitionEnd: function(ev, el, o) {
             el = Ext.get(el);
    -        var style = el.dom.style,
    +        var style = (el.dom || {}).style, // cdonnelly 2010-10-19: el.dom could be null at this point.
                 config = o.config,
                 property,
                 me = this;
    @@ -439,4 +459,4 @@
                 };
             }
         })
    -};
    \ No newline at end of file
    +};

  2. #2
    Ext JS Premium Member SimonFlack's Avatar
    Join Date
    Jul 2010
    Location
    Norway
    Posts
    241
    Vote Rating
    3
    SimonFlack is on a distinguished road

      0  

    Default


    Thanks... Been debugging this for 2 days.

  3. #3
    Sencha User benwhite's Avatar
    Join Date
    Oct 2010
    Location
    Columbus, OH
    Posts
    49
    Vote Rating
    0
    benwhite is on a distinguished road

      0  

    Default


    I was able to correct this issue very easily...

    Here are the two snipets of code I changed



    Ext.Anim.run
    from:
    Code:
                // Bind our listener that fires after the animation ends
                el.on('webkitTransitionEnd', me.onTransitionEnd, me, {
                    single: true,
                    config: config,
                    after: after
                });
    to:
    Code:
                // Bind our listener that fires after the animation ends
                el.on('webkitTransitionEnd', me.onTransitionEnd, me, {
                    single: true,
                    config: config,
                    after: after,
                    id: el.id
                });


    Ext.Anim.onTransitionEnd
    from:
    Code:
            el = Ext.get(el);
    to:
    Code:
            el = Ext.get(o.id);
    Hope this helps...

Similar Threads

  1. [OPEN-467] Nested List dont fire any event on backtap
    By jorslb in forum Sencha Touch 1.x: Bugs
    Replies: 2
    Last Post: 11 Apr 2011, 5:28 AM
  2. [FIXED-360] scroller.scrollTo doesn't fire scroll event
    By jeroenvduffelen in forum Sencha Touch 1.x: Bugs
    Replies: 7
    Last Post: 26 Nov 2010, 10:08 AM
  3. Replies: 4
    Last Post: 23 Jul 2010, 4:51 AM
  4. Replies: 11
    Last Post: 17 Feb 2010, 4:13 AM
  5. Fire event when region is open
    By Stephan123 in forum Ext 3.x: Help & Discussion
    Replies: 6
    Last Post: 24 Aug 2009, 4:39 AM

Thread Participants: 2

Tags for this Thread

Turkiyenin en sevilen filmlerinin yer aldigi xnxx internet sitemiz olan ve porn sex tarzi bir site olan mobil porno izle sitemiz gercekten dillere destan bir durumda herkesin sevdigi bir site olarak tarihe gececege benziyor. Sitenin en belirgin ozelliklerinden birisi de Turkiyede gercekten kaliteli ve muntazam, duzenli porno izle siteleri olmamasidir. Bu yuzden iste. Ayrica en net goruntu kalitesine sahip adresinde yayinlanmaktadir. Mesela diğer sitelerimizden bahsedecek olursak, en iyi hd porno video arşivine sahip bir siteyiz. "The Best anal porn videos and slut anus, big asses movies set..." hd porno faketaxi