Thank you for reporting this bug. We will make it our priority to review this report.
  1. #1
    Sencha - Ext JS Dev Team Animal's Avatar
    Join Date
    Mar 2007
    Location
    Notts/Redwood City
    Posts
    30,545
    Vote Rating
    64
    Animal is a jewel in the rough Animal is a jewel in the rough Animal is a jewel in the rough

      0  

    Default [FIXED][3.x] local var isIE6 not defined in core/core/Ext.js

    [FIXED][3.x] local var isIE6 not defined in core/core/Ext.js


    The code has

    Code:
    (function(){
        var idSeed = 0,
            ua = navigator.userAgent.toLowerCase(),
            check = function(r){
                return r.test(ua);
            },
            isStrict = document.compatMode == "CSS1Compat",
            isOpera = check(/opera/),
            isChrome = check(/chrome/),
            isWebKit = check(/webkit/),
            isSafari = !isChrome && check(/safari/),
            isSafari3 = isSafari && check(/version\/3/),
            isSafari4 = isSafari && check(/version\/4/),
            isIE = !isOpera && check(/msie/),
            isIE7 = isIE && check(/msie 7/),
            isIE8 = isIE && check(/msie 8/),
            isGecko = !isWebKit && check(/gecko/),
            isGecko3 = isGecko && check(/rv:1\.9/),
            isBorderBox = isIE && !isStrict,
            isWindows = check(/windows|win32/),
            isMac = check(/macintosh|mac os x/),
            isAir = check(/adobeair/),
            isLinux = check(/linux/),
            isSecure = /^https/i.test(window.location.protocol);
    
        // remove css image flicker
        if(isIE6){
            try{
                document.execCommand("BackgroundImageCache", false, true);
            }catch(e){}
        }
    That var ieIE6, is not defined at that point.

    It needs to be

    Code:
    (function(){
        var idSeed = 0,
            ua = navigator.userAgent.toLowerCase(),
            check = function(r){
                return r.test(ua);
            },
            isStrict = document.compatMode == "CSS1Compat",
            isOpera = check(/opera/),
            isChrome = check(/chrome/),
            isWebKit = check(/webkit/),
            isSafari = !isChrome && check(/safari/),
            isSafari3 = isSafari && check(/version\/3/),
            isSafari4 = isSafari && check(/version\/4/),
            isIE = !isOpera && check(/msie/),
            isIE7 = isIE && check(/msie 7/),
            isIE8 = isIE && check(/msie 8/),
            isIE6 = isIE && !isIE7 && !isIE8,
            isGecko = !isWebKit && check(/gecko/),
            isGecko3 = isGecko && check(/rv:1\.9/),
            isBorderBox = isIE && !isStrict,
            isWindows = check(/windows|win32/),
            isMac = check(/macintosh|mac os x/),
            isAir = check(/adobeair/),
            isLinux = check(/linux/),
            isSecure = /^https/i.test(window.location.protocol);
    
        // remove css image flicker
        if(isIE6){
            try{
                document.execCommand("BackgroundImageCache", false, true);
            }catch(e){}
        }
    
        ...
    
        isIE6 : isIE6,
    
        ...

  2. #2
    Sencha - Ext JS Dev Team evant's Avatar
    Join Date
    Apr 2007
    Location
    Sydney, Australia
    Posts
    17,159
    Vote Rating
    674
    evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute

      0  

    Default


    Fixed, thanks.
    Evan Trimboli
    Sencha Developer
    Twitter - @evantrimboli
    Don't be afraid of the source code!

Thread Participants: 1