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 8 fixes #34

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from
Open

iOS 8 fixes #34

wants to merge 8 commits into from

Conversation

tougher
Copy link

@tougher tougher commented Sep 25, 2014

My girlfriend was not happy about the iOS 8 update did break the app, because all her students uses it in all her classes :-)

So here you have a pull request. The most important change was in Brushes-Info.plist where public.data is changed to public.folder to make the app able to save/read data (iOS 8 does not understand public.data for a bundled folder). I also checked Full Screen at Launch in Interface Builder for the MainWindow.xib to make the view work in iOS 8. For some reason it was necessary to import OpenGLES/ES2/gl.h in WDUtilities.h to make it build.

There seems to be a lot of number conversion precision issues when building for 64 bit, but I did just set the compiler to ignore it.

I've tested on an iPad 1 (5.1.1), iPad Air (8.0) and the iOS 7.1 simulator.

@tougher
Copy link
Author

tougher commented Sep 26, 2014

I forgot that the app also works on iPhone. The MainWindow~iphone.xib may need Full Screen at Launch as well.

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.

2 participants