I went to surf my site on the iPad tonight at the Apple store, and realized that it's being re-directed to the mobile site, which is NOT what I want. Can we add an exception for that version of Safari? Thanks. Love the module, but need to fix this.

CommentFileSizeAuthor
#7 mobile_tools-added_iPad_group-767796.patch624 bytescrookednumber

Comments

tandersonmd’s picture

tandersonmd commented

I'm very interested in this as well.

Thomas

pribeh’s picture

pribeh commented

From http://developer.apple.com/safari/library/technotes/tn2010/tn2262.html#T...

Mozilla/5.0 (iPad; U; CPU OS 3_2 like Mac OS X; en-us) AppleWebKit/531.21.10 (KHTML, like Gecko) Version/4.0.4 Mobile/7B334b Safari/531.21.10

twom’s picture

twom commented
Assigned: Unassigned » twom
Status: Active » Fixed

the new dev version has this fixed!

pribeh’s picture

pribeh commented

amazing.

efaerber’s picture

efaerber commented

Looks like you have a typo on line 496 in mobile_tools.module. "reak" instead of "break"

twom’s picture

twom commented

oops... Being committed right now!

crookednumber’s picture

crookednumber commented
Status: Fixed » Patch (to be ported)
StatusFileSize
new mobile_tools-added_iPad_group-767796.patch624 bytes

Seems like it would make sense to add iPad to _mobile_tools_device_groups() as well. Yes?

Patch attached.

twom’s picture

twom commented

great idea. Patch is committed.

Tom

twom’s picture

twom commented
Category: bug » feature
Priority: Critical » Normal
Status: Patch (to be ported) » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

ylc208’s picture

ylc208 commented
Version: 6.x-1.x-dev » 6.x-2.1
Assigned: twom » Unassigned

This patch didn't work for my ipad.
Can you work on another one? Thank you.