Skip to content
This repository has been archived by the owner on Oct 7, 2020. It is now read-only.

Fix for iPhone devices 6 and 6+ #135

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

AlejandroFernandesAntunes

Description:

  • iPhone devices are reporting:
    "Mozilla/5.0 (iPhone; CPU iPhone OS 9_1 like Mac OS X) AppleWebKit/601.1.46 (KHTML, like Gecko) Version/9.0 Mobile/13B143 Safari/601.1"

when UA is retrieved, so just asking for Safari will do. We don't want the user to open chrome on an iPhone and get sent to the Android PlayStore


@jdalton
Copy link
Contributor

jdalton commented Jul 14, 2016

@AlejandroFernandesAntunes

Ah, UA sniffs.
You should update the MDN wiki.

There is a note in the existing code:

// Check webview and native smart banner support (iOS 6+).

I take that as a todo?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants