-
Notifications
You must be signed in to change notification settings - Fork 154
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
Project modernisation #32
base: master
Are you sure you want to change the base?
Conversation
Hei that's good! And moving to ARC is a must, it's true. The only thing is that your ARC changes makes PrettyKit no longer compatible with 4.0. In my opinion, once we break 4.0 compatibility, we should start using UIAppearance. That would be quite a big change, and I would like to support 4.0 a bit more, so what if the change to ARC is made still supporting 4.0? (weak attribute can't be used, for example) |
The Storyboard itself limits compatibility to iOS 5+. But the library itself can be kept compatible with iOS 4, true. |
But your modifications don't force the use of storyboards, do them? I mean, what you changed was implementing initWithCoder, right? And that doesn't require iOS 5 at all, so the lib would be still compatible with iOS 4. |
Yes, that's exactly what I said: “the library itself can be kept compatible with iOS 4” :) And that's what I just fixed, along with a couple other outstanding issues :) |
up :) |
Is this ever going to get pulled into the project? |
It will, eventually. I don't have enough time to update pretty kit, but I In the meantime it will be better that you use your own fork instead. On 19 Xan, 2013, at 23:25, Bradley Ringel notifications@github.com wrote: Is this ever going to get pulled into the project? — |
iPhone 5 & iOS 6 compatibility, ARC, Storyboards.
Includes irrationalfab's patch from Jul 30th, 2012 and a full Storyboard example of my own.