New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't work under IE8 #1294

Closed
keepallsimple opened this Issue Mar 26, 2011 · 5 comments

Comments

Projects
None yet
3 participants
@keepallsimple

keepallsimple commented Mar 26, 2011

Hi, i use git clone the latest jquery-mobile, and it does't work, the page not show, so, i use the IE8 internal debuger found that exception in jquery.mobile-1.0a4pre.js at line 1828 :

//kill the keyboard
$( window.document.activeElement ).add( "input:focus, textarea:focus, select:focus" ).blur();

so, i think comments this line, then try again, well the firt page is visible, but the CPU up to 100%, the system is blocked, i had to kill the IE.

can any one know why ?

PS: under IE8, the site http://jquerymobile.com/demos/1.0a3/ not work, only show the raw html.

@scottjehl

This comment has been minimized.

Show comment
Hide comment
@scottjehl

scottjehl Mar 26, 2011

Contributor

sounds like it's not supported. Can you try something like this for us?

   $( document.activeElement || "" ).add( "input:focus, textarea:focus, select:focus" ).blur();

Thanks!

Contributor

scottjehl commented Mar 26, 2011

sounds like it's not supported. Can you try something like this for us?

   $( document.activeElement || "" ).add( "input:focus, textarea:focus, select:focus" ).blur();

Thanks!

@keepallsimple

This comment has been minimized.

Show comment
Hide comment
@keepallsimple

keepallsimple Mar 26, 2011

Hi, scottjehl, i try your way, but still not work.
and found that broken at jquery-1.5.2rc1.js line 4039:

filter: {
    PSEUDO: function( elem, match, i, array ) {
        var name = match[1],
            filter = Expr.filters[ name ];

        if ( filter ) {
            return filter( elem, i, match, array );

        } else if ( name === "contains" ) {
            return (elem.textContent || elem.innerText || Sizzle.getText([ elem ]) || "").indexOf(match[3]) >= 0;

        } else if ( name === "not" ) {
            var not = match[3];

            for ( var j = 0, l = not.length; j < l; j++ ) {
                if ( not[j] === elem ) {
                    return false;
                }
            }

            return true;

        } else {
            Sizzle.error( name ); <<============== line 4039
        }
    },

and from the IE internal debugger ,i found the variable name's value is "focus", and from the code , we can see that PSEUDOnot support 'foucs' .

can anyone give help?

keepallsimple commented Mar 26, 2011

Hi, scottjehl, i try your way, but still not work.
and found that broken at jquery-1.5.2rc1.js line 4039:

filter: {
    PSEUDO: function( elem, match, i, array ) {
        var name = match[1],
            filter = Expr.filters[ name ];

        if ( filter ) {
            return filter( elem, i, match, array );

        } else if ( name === "contains" ) {
            return (elem.textContent || elem.innerText || Sizzle.getText([ elem ]) || "").indexOf(match[3]) >= 0;

        } else if ( name === "not" ) {
            var not = match[3];

            for ( var j = 0, l = not.length; j < l; j++ ) {
                if ( not[j] === elem ) {
                    return false;
                }
            }

            return true;

        } else {
            Sizzle.error( name ); <<============== line 4039
        }
    },

and from the IE internal debugger ,i found the variable name's value is "focus", and from the code , we can see that PSEUDOnot support 'foucs' .

can anyone give help?

@johnbender

This comment has been minimized.

Show comment
Hide comment
@johnbender

johnbender Apr 2, 2011

Contributor

activeElement conditional for windows support Closed by 5c37bea

Contributor

johnbender commented Apr 2, 2011

activeElement conditional for windows support Closed by 5c37bea

@johnbender johnbender closed this Apr 2, 2011

@keepallsimple

This comment has been minimized.

Show comment
Hide comment
@keepallsimple

keepallsimple Apr 2, 2011

hi, johnbender thanks much!

i like jqueymobile!

keepallsimple commented Apr 2, 2011

hi, johnbender thanks much!

i like jqueymobile!

@johnbender

This comment has been minimized.

Show comment
Hide comment
@johnbender

johnbender Apr 5, 2011

Contributor

keepallsimple,

Glad to hear it and thanks for taking the time to submit the issue!

Contributor

johnbender commented Apr 5, 2011

keepallsimple,

Glad to hear it and thanks for taking the time to submit the issue!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment