Results 1 to 2 of 2

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

    Thank you for reporting this bug. We will make it our priority to review this report.
  1. #1
    Sencha User Animal's Avatar
    Join Date
    Mar 2007
    Location
    Bédoin/Redwood City
    Posts
    30,752
    Vote Rating
    83
      0  

    Default [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,
    
        ...
    Longtime Sencha engineer. Now surplus to requirements apparently...

  2. #2
    Sencha User evant's Avatar
    Join Date
    Apr 2007
    Location
    Sydney, Australia
    Posts
    18,965
    Vote Rating
    931
      0  

    Default

    Fixed, thanks.
    Twitter - @evantrimboli
    Former Sencha framework engineer, available for consulting.
    As of 2017-09-22 I am not employed by Sencha, all subsequent posts are my own and do not represent Sencha in any way.

Posting Permissions

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