Skip to main content

Bug Tracker

Side navigation

#3179 closed enhancement (fixed)

Opened July 23, 2008 03:16PM UTC

Closed July 23, 2008 03:21PM UTC

Last modified August 13, 2009 09:23AM UTC

jQuery.browser.version fallback

Reported by: flesler Owned by: flesler
Priority: minor Milestone: 1.3
Component: core Version: 1.2.6
Keywords: Cc:
Blocked by: Blocking:
Description

Hi, I noted that on jQuery, when it calculates

jQuery.browser.version and not matches Mozilla,

MSIE, Opera or Webkit, it puts the undefined value.

That would of no harm, but Digg.com (on digg.js)

uses this value as version.split('.'), so

non-recognized browsers trigger a JS error, my

suggestion is to change [])[1] to at least:

[0,"1.0"])[1]

or:

userAgent.match(/\\/([\\d.]+)/))[1]

So 'version' gets assigned a meaningful value.

This would benefit new browsers.

Óscar Toledo G.

Attachments (0)
Change History (2)

Changed July 23, 2008 03:21PM UTC by flesler comment:1

need: PatchCommit
owner: → flesler
status: newassigned

Changed July 23, 2008 03:21PM UTC by flesler comment:2

resolution: → fixed
status: assignedclosed

Added at [5791].