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

Master #41

Open
wants to merge 32 commits into
base: objective-c
Choose a base branch
from
Open

Master #41

wants to merge 32 commits into from

Conversation

githubleez
Copy link

Fix Xcode 7.3 selector error and swift warnings

smileyborg and others added 30 commits June 7, 2014 21:59
In case the cell is too tall, better to have the extra space go here.
These are issues with Xcode 6b1 and the iOS 8 SDK first seed.
By making the (inferred type) Double constants an explicit CGFloat
type, they can be used as expected.
Resolves #2. Thanks to @mbogh for helping put this together!

There are still 2 outstanding issues, they seem like Apple bugs at this point:
1. Cells loaded from a nib do not size correctly in iOS 8 when the table view width is not 320pts (e.g. landscape iPhone, any iPad). (Programmatically loaded cells work fine.)
2. Changing the Dynamic Type slider while the app is running does not fire the UIContentSizeCategoryDidChangeNotification, meaning the app does not respond by updating its UI until it is killed and launched again.
Changes to fix compiler errors with beta 3 and 4
…ed launch images for 4", 4.7", and 5.5" phones, set NibTableViewCell's bodyLabel preferred width to automatic.
Fixes #21
- Avoid overriding any of the table view controller initializers
- Refactor and modernize the Model class
All these changes are backwards-compatible with earlier Swift versions.
- Remove manual integration of PureLayout source
- Set up CocoaPods for this project (Pods are checked in to avoid
requiring `pod install` when cloning the project)
- Upgrade to PureLayout v3.0.0
- Changes required for v3.0.0 of PureLayout
The original .gitignore was outdated, and was preventing the workspace from being added to the repo.
cl7 and others added 2 commits May 14, 2016 23:15
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.

None yet

6 participants