Ticket #3402 (closed bug: duplicate)
jQuery.browser.version can be incorrect with mutant IE userAgent strings
| Reported by: | dbergey | Owned by: | flesler |
|---|---|---|---|
| Priority: | major | Milestone: | 1.3 |
| Component: | core | Version: | 1.2.6 |
| Keywords: | browser version msie | Cc: | |
| Blocking: | Blocked by: |
Description
jquery.browser.version currently looks at the last match of /ie ([\d.]+)/ .. it should look for all matches, then use the highest found version. In some situations IE 7 will present itself as both IE 7.0 and IE 6.0 for compatibility, and sometimes the declarations get out of order, not to mention adware and toolbars messing with the registry (see: http://jamazon.co.uk/web/2008/07/23/an-ie7-bug-that-returns-msie-60-user-agent-string/). The attached patch supplements the existing userAgent checking, but could be improved upon.
Attachments
-
ieversion.patch
(360 bytes) -
added by dbergey 5 years ago.
-
ieversion2.patch
(243 bytes) -
added by dbergey 5 years ago.
- Updated patch (fewer lines, future-proofed)
Change History
comment:1 Changed 5 years ago by flesler
- Owner set to flesler
- Status changed from new to assigned
There were already more than 1 tickets for this, with no 100% successful patch.
I'll keep this ticket in mind for when we handle this.
comment:2 Changed 5 years ago by dbergey
Updated patch to be IE-version-agnostic, and reduced from 7 to 2 lines.
Changed 5 years ago by dbergey
-
attachment
ieversion2.patch
added
Updated patch (fewer lines, future-proofed)
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.