On Tuesday 03 July 2007, John Resig wrote:
> I'm confused - are you saying the Konquerer breaks because of the
> regexp, or that the regexp breaks in Konquerer?
The regexp
/.+(?:rv|it|ra|ie)[\/: ]([\d.]+)/
itself is fine, but it doesn't match
konqueror/3.5
Therefore
b.match(/.+(?:rv|it|ra|ie)[\/: ]([\d.]+)/)[1]
is equivalent to
null[1]
which fails and stops any further execution of the script.
Michael
> On 7/2/07, Michael Schuerig <[EMAIL PROTECTED]> wrote:
> > Sorry to report this here, but the site appears to be largely
> > non-functional due to the popularity of the recent release of
> > jQuery 1.1.3.
> >
> > Unfortunately, 1.1.3 breaks completely on Konqueror, at least in my
> > case where the lower-cased user agent string looks like this
> >
> > mozilla/5.0 (compatible; konqueror/3.5; linux) khtml/3.5.7 (like
> > gecko) (debian)
> >
> > This string doesn't result in a match for the browser version when
> > the original pattern is used
> >
> > jQuery.browser = {
> > version: b.match(/.+(?:rv|it|ra|ie)[\/: ]([\d.]+)/)[1],
> > ...
> > };
> >
> > The fix is easy:
> >
> > jQuery.browser = {
> > version: b.match(/.+(?:rv|it|ra|ie|or)[\/: ]([\d.]+)/)[1],
> > ...
> > };
--
Michael Schuerig
mailto:[EMAIL PROTECTED]
http://www.schuerig.de/michael/
