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

iOS version of IITCm #1037

Open
wants to merge 80 commits into
base: master
Choose a base branch
from
Open

Conversation

HubertZhang
Copy link

I've ported IITCm to iOS, however there is still some bugs and some features of IITCm is missing..
I won't have too much time on this, thus I decide to open this pull request.

This pull request is based on #1025 , which add detection for iOS devices
And the main idea of the App is use WKWebView and inject JS codes when loading intel. I wrote a 'hook' which translate from call to 'android' functions to a Message Handlers, thus JS codes can interact with App.

Some screen shots can be found below.

@Cojad
Copy link

Cojad commented Aug 15, 2015

impressive!!

@umer936
Copy link

umer936 commented Aug 15, 2015

Looks pretty nice!
However, I did want to ask what the benefits of this over something like the bookmarklet I made ( #833 ) are, other than this being a native app.

Speed improvements? Better navigation? Also, do you load the scripts locally or pull the JS from the server each time?

Thanks!

@mxxcon
Copy link

mxxcon commented Aug 16, 2015

@umer936 does iOS still cripple not-Safari JS performance? If so then I think your bookmarklet code would still be faster. Looks like this app's 'native' settings menus are easier to use.

@HubertZhang
Copy link
Author

@mxxcon According to WWDC 2014, WKWebView uses the same JS engine as Safari. Thus I think the app's performance won't be too bad.

@HubertZhang
Copy link
Author

@umer936 The scripts are loaded locally currently. And I'm planning to allow user upload their own scripts for debug and update scripts from server. But I may not have enough time...

@michaeldever
Copy link
Contributor

Nice, very impressive.

@flamusdiu
Copy link

@HubertZhang http://ingressgps.com/? Is this based on your code or something different?

@HubertZhang
Copy link
Author

@flamusdiu No...

@flamusdiu
Copy link

@HubertZhang rgr. This was brought to my attention. Umm ...

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

Successfully merging this pull request may close these issues.

6 participants