Opened 7 years ago

Closed 7 years ago

#16938 closed defect (fixed)

dojox.mobile.sniff reports true for has('bb') in Safari/OSX

Reported by: johan Owned by: Eric Durocher
Priority: undecided Milestone: 1.8.4
Component: DojoX Mobile Version: 1.8.4rc1
Keywords: Cc:
Blocked By: Blocking:

Description

On a page I use dojox.mobile.sniff to check for mobiles. The issue is that the statement has('bb') returns true when I access the page from my Macbook (OS X 10.8.2) using Safari Version 6.0.2 (8536.26.17).

Attachments (2)

fix_sniff_bb10.patch (600 bytes) - added by Eric Durocher 7 years ago.
Fix BlackBerry 10 sniff test (trunk) - Eric Durocher (IBM, CCLA)
fix_sniff_bb10_1.8.patch (664 bytes) - added by Eric Durocher 7 years ago.
Fix BlackBerry 10 sniff test (1.8) - Eric Durocher (IBM, CCLA)

Download all attachments as: .zip

Change History (7)

Changed 7 years ago by Eric Durocher

Attachment: fix_sniff_bb10.patch added

Fix BlackBerry 10 sniff test (trunk) - Eric Durocher (IBM, CCLA)

Changed 7 years ago by Eric Durocher

Attachment: fix_sniff_bb10_1.8.patch added

Fix BlackBerry 10 sniff test (1.8) - Eric Durocher (IBM, CCLA)

comment:1 Changed 7 years ago by Eric Durocher

Wow, good catch! missing >= 0 test... Thanks!

comment:2 Changed 7 years ago by cjolif

In [31119]:

refs #16938. Fix BlackBerry? 10 sniff test (trunk) - Eric Durocher (IBM, CCLA)

comment:3 Changed 7 years ago by cjolif

In [31120]:

refs #16938. Fix BlackBerry? 10 sniff test (1.8) - Eric Durocher (IBM, CCLA)

comment:4 Changed 7 years ago by cjolif

Milestone: tbd1.8.4

comment:5 Changed 7 years ago by cjolif

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.