Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#3669 closed patch (fixed)

Updates for mobile browser detection on Windows Mobile devices

Reported by: skd5aner@… Owned by: xris
Priority: minor Milestone: unknown
Component: mythweb Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

The attached patch includes the following updates:

  • Adds current versions of Opera Mobile (8.60 and 8.65). Windows Mobile will not run Opera Mini natively (it's Java). I am not sure if "Opera 8.65" and "Opera 8.60" might coincide with any desktop versions and incorrectly identify them. If so, at least on Windows Mobile 5, the catch-all mentioned below will still work for these browsers.
  • Adds Minimo - The Mozilla firefox mobile browser for multiple platforms (including windows mobile).
  • Adds a generic catch-all for anything passing "Windows CE" in the header information. All three browsers (Opera Mobile, Pocket IE, and Minimo) passed "Windows CE" in the header information. If for some reason the are not explicitly caught (new version, etc), the "Windows CE" will succesfully select the mobile theme. I tested by commenting out the Opera and Minimo options and the catch-all succesfully selected the mobile theme for all browsers. I have not tested any other browsers, but if they pass "Windows CE" this catch-all should work for them as well.

Reasoning: Opera Mobile will support Digest Authentication whereas Pocket IE will not. I could not get Minimo to work properly while passing authentication, but it worked when I disabled authentication on apache for testing purposes. Pocket IE successfully worked when authentication was turned off as well.

Matt

Attachments (1)

wm-mobile.patch (1.4 KB) - added by skd5aner@… 13 years ago.

Download all attachments as: .zip

Change History (4)

Changed 13 years ago by skd5aner@…

Attachment: wm-mobile.patch added

comment:1 Changed 13 years ago by xris

(In [13806]) Update mobile browser detection re #3669

comment:2 Changed 13 years ago by xris

Resolution: fixed
Status: newclosed

Sorry, can't add the other two opera strings because of possible conflicts with non-mobile browsers..

comment:3 in reply to:  2 Changed 13 years ago by skd5aner@…

Replying to xris:

Sorry, can't add the other two opera strings because of possible conflicts with non-mobile browsers..

That's OK, because if the Opera browser is used on a Windows Mobile device, it should still be caught by the catch-all since Opera sends the "windows CE" tag in the header info. Therefore, it should still utilize the mobile theme.

Funny thing though, I just cancelled my mobile data plan, so I can't test it any longer. Hope it benefits everyone else though! =D

Thanks! Matt

Note: See TracTickets for help on using tickets.