-
-
Notifications
You must be signed in to change notification settings - Fork 487
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
IE11 on Windows 8.1 can cause tablet
to be true
#89
Comments
I have face this issue and to be more specific the false positive is on laptops IE10+ with touch capability. While for non touch laptops (which works fine) is: Mozilla/5.0 (compatible; MSIE 10.0; Windows NT 6.2; WOW64; Trident/6.0) Cheers, Sergio |
👍 |
I could verify this - - on a dual monitor no touch desktop pc. |
@digital-void, thanks, and what kind of Wacom Tablet do you use? What about UA in that case? It would be helpful if you send to us browser's UA :) |
Wacom Intuos Pro - but I guess this will happen with all touch input devices on windows, as there is no difference between an external or monitor build in touch-input-device on the api side. Same thing happening on my Chintiq Companion - it's no touch screen monitor - but a monitor with a build in stylus system - that is touch capable... And - - - what do you mean with UA? switch(bowser.agent){
case ie:
doSomeIeSpecial();
break;
case chrome:
doSomeChromeSpecial();
break;
default:
doDefault();
}; cheers - |
@digital-void, I meant user agent string, that available as And yeah, I've thought about that case — add something like |
sure: |
@digital-void, interesting. Thank you. I'll test it ASAP and I'm thinking about that feature with additional flags for platform. |
Actually - the platform is only a part of it - heaps of people use Firefox 32bit - event though they're on an 64bits os... |
btw - your support is outer standing ;) |
Hi, |
Hi, @shaibv! Actually, not yet. Do you have the same problem? Describe it, please. |
Hi, Yes, I have the same problem. |
Hi, Yes, I have the same problem. Thanks, On Tue, Oct 25, 2016 at 11:55 AM, Denis Demchenko [email protected]
|
Could you, please, provide your
|
Mozilla/5.0 (Windows NT 10.0; WOW64; Trident/7.0; Touch; LCTE; rv:11.0) |
Hi @lancedikson Do you have any idea how can I work around this issue? |
@shaibv, I put some tests for the devices that you've prevented above and tests are passed: https://travis-ci.org/ded/bowser/builds/172985408 |
Hi @lancedikson , |
@shaibv, any news about that? :) |
Hi, On Wed, Nov 9, 2016 at 1:07 PM, Denis Demchenko [email protected]
|
+1, same here, desktop PC windows 10 with Wacom Cintiq attached. "Mozilla/5.0 (Windows NT 10.0; WOW64; Trident/7.0; .NET4.0C; .NET4.0E; .NET CLR 2.0.50727; .NET CLR 3.0.30729; .NET CLR 3.5.30729; Tablet PC 2.0; rv:11.0) like Gecko" bowser.tablet === true Doesn't happen with MS Edge: "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.79 Safari/537.36 Edge/14.14393" bowser.tablet is undefined in that case |
@jakedowns, thanks for the report. It's a known case. But, unfortunately, I can't suggest you any workaround for it. Bowser just makes some guesses using UserAgent string and it could be wrong because of those cases, when a tablet attached to PC. |
that's fair. thanks for the prompt feedback and a great utility. been using it for years on all our projects
… On Dec 16, 2016, at 5:21 PM, Denis Demchenko ***@***.***> wrote:
@jakedowns, thanks for the report. It's a known case. But, unfortunately, I can't suggest you any workaround for it. Bowser just makes some guesses using UserAgent string and it could be wrong because of those cases, when a tablet attached to PC.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
I guess, we can close this for now, since there is no clear solution for it. |
Checking for
mobile
andtablet
can potentially cause false positives on certain Windows 8 devices (in our case a Asus X551M but users reported issues from MS Surface's as well).We did not catch this using the MS provided VM's.
I realize that this is not technically a bug in bowser but I wanted to put this out there in case anybody else runs into the same issue.
To get around this we are now doing the following:
The text was updated successfully, but these errors were encountered: