Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Nintendo DS User Agent in mobile #1695

Closed
StevenWeathers opened this Issue · 2 comments

3 participants

@StevenWeathers

I recently grabbed the latest user_agents config to update for android and others, and had some issues with IE8.
Apparently the User Agent string includes MDDS for some machines, not sure what that really is but the underlying issue I had was that 'ds' => 'Nintendo DS' was causing my clients site to show the mobile version for some IE8 users. I've commented this out as we really don't care about Nintendo DS users (i don't think they've even had 1). Perhaps you guys should look at this, as I doubt many sites support/have nintendo ds visitors but do have plenty of IE8 users.

-Thanks

@cryode

Do you have the exact user agent string that contains MDDS? Might help for reference.

Based on all the info I can find regarding DS user agents, it seems that adding "nintendo" to the array key would be sufficient. Any agents that didn't include "nintendo" had Opera Mini defined, which will still toggle as mobile.

@StevenWeathers

Just incase anyone wanted it heres the user agent string that had the issue.

Mozilla/4.0 (compatibile; MSIE 8.0; Windows NT 6.1; WOW64; Trident/4.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0; MDDS; .NET4.0C; .NET4.0E)
@cryode cryode referenced this issue from a commit
@cryode cryode Remove change log entry for Bug fix #1695
Signed-off-by: Eric Roberts <eric@cryode.com>
c58c782
@narfbg narfbg closed this issue from a commit
@cryode cryode Bug fix #1695 - change Nintendo mobile user agents to be more specific.
Signed-off-by: Eric Roberts <eric@cryode.com>
19d0f56
@narfbg narfbg closed this in 19d0f56
@nonchip nonchip referenced this issue from a commit in nonchip/CodeIgniter
@cryode cryode Bug fix #1695 - change Nintendo mobile user agents to be more specific.
Signed-off-by: Eric Roberts <eric@cryode.com>
6cba1f8
@nonchip nonchip referenced this issue from a commit in nonchip/CodeIgniter
@cryode cryode Remove change log entry for Bug fix #1695
Signed-off-by: Eric Roberts <eric@cryode.com>
e4fd9de
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.