#2661 closed bug (fixed)
Browser detection IE (and possibly other browsers)
Reported by: | cybernetix | Owned by: | flesler |
---|---|---|---|
Priority: | major | Milestone: | 1.4 |
Component: | core | Version: | 1.4a1 |
Keywords: | browser detection | Cc: | |
Blocked by: | Blocking: |
Description
If MEGAUPLOAD or similar plugin is installed jQuery.browser returns wrong IE version.
For example userAgent string for IE7 with installed MEGAUPLOAD plugin is:
Mozilla/4.0 (compatible; MSIE 7.0; Windows NT 5.1; Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; SV1) ; .NET CLR 2.0.50727; .NET CLR 1.1.4322; .NET CLR 3.0.04506.30; MEGAUPLOAD 2.0)
jQuery.browser.version returns 6 instead of 7
First plus (+) should be removed from regExp and than it will work fine. So, this line:
[])[1] |
should read:
[])[1] |
Change History (16)
comment:1 Changed 15 years ago by
Owner: | set to flesler |
---|---|
Status: | new → assigned |
comment:2 Changed 15 years ago by
Milestone: | 1.2.3 → 1.2.4 |
---|
comment:3 Changed 15 years ago by
The solution you posted doesn't actually work with Opera. Here's one that passes all the jQuery tests (thanks Brad Greenlee!):
/.+?(?:rv|it|ra|ie)[\/: ]([\d.]+)(?!.+opera)/
comment:5 Changed 14 years ago by
comment:6 Changed 14 years ago by
Has anyone looked into this? It's clearly related, and his patch seems to work...
http://jamazon.co.uk/web/2008/03/14/jquerybrowserversion-doesnt-recognise-ie7/
http://jamazon.co.uk/web/2008/03/17/thickbox-31-ie7-positioning-bug/
comment:8 Changed 14 years ago by
comment:14 Changed 14 years ago by
comment:16 Changed 13 years ago by
Milestone: | 1.2.4 → 1.4 |
---|---|
Resolution: | → fixed |
Status: | assigned → closed |
Version: | 1.2.3 → 1.4a1 |
Same as #2784