Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

should not display if in-app web viewer #16

Open
GoogleCodeExporter opened this issue Mar 20, 2015 · 0 comments
Open

should not display if in-app web viewer #16

GoogleCodeExporter opened this issue Mar 20, 2015 · 0 comments

Comments

@GoogleCodeExporter
Copy link
Collaborator

What steps will reproduce the problem?
1. Load the webapp within a web view inside some non-Safari app.
2. The bubble appears, pointing at a non-existent Bookmark button.

What is the expected output? What do you see instead?

The bubble should not appear if the user isn't viewing the web app with Mobile 
Safari. It does appear.


What version of the product are you using? On what operating system?

The version listed as revision b3d04bb1ed17 in the change log. Downloaded on 
July 25, 2011. Hosting on Mac OS X 10.6.7, and viewing on iOS 4.3.5. 


Please provide any additional information below.

The regex that tests for Safari actually only tests for a web view on an 
iPhone, iPod, or iPad. It should actually test for safari, since web views 
within non-Safari apps leave that out of the user agent (which is otherwise the 
same). 

SOLUTION: Change the regular expression for detecting Mobile Safari, 
MOBILE_SAFARI_USERAGENT_REGEX_, to the following:
/(iPhone|iPod|iPad).*Safari/

That works properly, at least when tested with a web app running within 
FileMaker Go versus the same web app running with Mobile Safari. Both return 
true when only testing for /iPhone|iPod|iPad/ 

Original issue reported on code.google.com by danielas...@gmail.com on 26 Jul 2011 at 9:15

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

No branches or pull requests

1 participant