[wp-trac] Re: [WordPress Trac] #8933: jQuery/Thickbox Issue IE7/8

[WordPress Trac]

#8933: jQuery/Thickbox Issue IE7/8
-------------------------------------+--------------------------------------
 Reporter:  alexleonard              |        Owner:     
     Type:  defect (bug)             |       Status:  new
 Priority:  normal                   |    Milestone:  2.8
Component:  JavaScript               |      Version:  2.7
 Severity:  normal                   |   Resolution:     
 Keywords:  has-patch needs-testing  |  
-------------------------------------+--------------------------------------
Comment (by Denis-de-Bernardy):

 there is also code here that may potentially be worth adding:

 http://jamazon.co.uk/web/2008/07/23/an-ie7-bug-that-returns-msie-60-user-
 agent-string/


 ----


 > A possible patch below redefines the browser version as 7.0 only if the
 browser has already been identified as IE6 and the the presence of the
 XMLHttpRequest object suggests otherwise.

 {{{
 if(jQuery.browser.msie && parseInt(jQuery.browser.version) == 6 &&
 this.XMLHttpRequest) { jQuery.browser.version = "7.0" }

 }}}

 > I’m thinking though that maybe this might not be bulletproof. If IE8
 presents the same or similar "Hi i'm IE6!" behaviour then the above patch
 could incorrectly identify IE8 as IE7. This might not be as serious as IE7
 vs. IE6 but it’s still an issue.

-- 
Ticket URL: <http://trac.wordpress.org/ticket/8933#comment:11>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software