[Tickets #6281] Why force iPhone to mimp/mobile mode?

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/?id=6281
-----------------------------------------------------------------------
 Ticket             | 6281
 Created By         | Eric Rostetter <eric.rostetter at physics.utexas.edu>
 Summary            | Why force iPhone to mimp/mobile mode?
 Queue              | Horde Framework Packages
 Version            | FRAMEWORK_3
 Type               | Enhancement
 State              | New
 Priority           | 1. Low
 Milestone          | 
 Patch              | 
 Owners             | 
-----------------------------------------------------------------------


Eric Rostetter <eric.rostetter at physics.utexas.edu> (2008-02-18 14:32)
wrote:

horde/framework/Browser/Browser.php forces the iPhone to mobile mode, hence
defaulting to mimp instead of imp.  While the iPhone is mobile, it has a
more-or-less full Safari which works fine in non-mobile mode.  My iPhone
users all want the imp and not mimp interface, and the mimp login screen
doesn't let them choose the interface the
way the imp login screen does.

I patched it with the following overly long patch to not mark the iPhone
as mobile:

494a495,501
>           if (strpos($agent, 'iPhone' ) !== false &&
>                 strpos($agent, 'Safari' ) !== false &&
>                 strpos($agent, 'Mobile' ) !== false ) {
>               // Exclude the iPhone -- EJR
>                 $this->_mobile = false;
>             }

It would be better probably to just except the "iPhone" pattern in the
"if" statement above
that though.

I don't know of any other implications of doing this for other apps.  No
one has reported any problems at least...

An alternate solution would be to allow the mimp login screen
(template/mobile.php I think) to allow them to select the interface
(traditional, dynamic, mobile), defaulting to mobile, so they could
overturn the default mail interface (if for some reason, the above change
isn't appropriate).