Skip to main content

Bug Tracker

Side navigation

#2418 closed bug (fixed)

Opened February 26, 2008 07:49PM UTC

Closed June 20, 2010 05:45PM UTC

Safari/WebKit version is checked incorrectly, causing null values

Reported by: mcrawford620 Owned by:
Priority: trivial Milestone: 1.2.4
Component: core Version: 1.2.3
Keywords: safari, webkit Cc:
Blocked by: Blocking:
Description

On line 1353, there is the test:

var chars = jQuery.browser.safari && parseInt(jQuery.browser.version) < 417

The bug you're working around was fixed in WebKit build 417.5. See:

http://trac.webkit.org/projects/webkit/browser/tags/JavaScriptCore-418/JavaScriptCore/ChangeLog

So I think the line 1353 test needs to be something like:

jQuery.browser.safari && jQuery.browser.version < 417.5

but I don't know exactly how the version comparison works (int, float, etc.).

Yes, I was forced to test my website on a WebKit build 417.2, and the jQuery code bombed. I don't have that build myself, it's on my client's machine, so I can't make a test case or anything like that. For our site, I changed the line to read parseInt(jQuery.browser.version) < 418 which made our client happy, but I guess that's not technically correct either.

Attachments (0)
Change History (1)

Changed June 20, 2010 05:45PM UTC by dmethvin comment:1

resolution: → fixed
status: newclosed

Browser check has been removed.