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
$element.is(':focus') does not work on mobile Chrome (maybe Webkits in general) #385
Comments
|
Looks like Chome fixed this bug at some point but the Webkit-based browsers still have it? Lines 705 to 709 in b05a9d3
|
|
I have Chrome 43 installed on my test phone. It does not seem to be fixed in it. |
|
Interesting, it works on desktop Chrome 52. In any case, that comment explains why it can't be fixed in Sizzle without causing IE8/9 to throw errors, which is worse. |
|
Yeah, it is not problem on desktop browsers. So we leave this broken for some browsers? No workaround or double check here? |
|
We'd need to know what to check and do a proper support test. The problem is we can't test focus-related things because we'd have to hijack the focus of the whole page and that's too disruptive. Yes, there are cases that we're unable to workaround because there are no reliable detection ways and we try to avoid user agent sniffing. If we know that the IE 8/9 issue is really only applicable to IE<10 (what about IE 6/7?), we might need to do an ugly check for |
|
That's too bad. So the only way to avoid this issue is on the developers end by checking for |
|
@func0der Sizzle has broader browser support than jQuery; see https://github.com/jquery/sizzle/wiki#-browser-support. jQuery browser support is available at https://jquery.com/browser-support/. |
|
Yeah, that's weird we still want to support those browsers :/. I'm sure @gibson042 has some reasons for it though. My vote is just to drop it |
It seems that the $element.is(':focus') detection of jQuery and therefore Sizzle has troubles detecting the currently focused element in the Android version of the Chrome browser.
I have created fiffdle for this problem: https://jsfiddle.net/abcqLoaL/1/
I had the same issue in Safari on an iOS device. Maybe this is a webkit related problem.
Firefox works fine though. Desktop browser seem to handle it fine.
The text was updated successfully, but these errors were encountered: