-
Notifications
You must be signed in to change notification settings - Fork 1
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
Update to latest master + unselected color + pod/spm support #5
base: master
Are you sure you want to change the base?
Conversation
We need to place subviews onto UITableViewCell's contentView (to use `UITableViewCell.insetsContentViewsToSafeArea`).
by swift 4.1 syntax * UIColor.whiteColor() -> UIColor.white * UIFont.systemFontOfSize(18) -> UIFont.systemFont(ofSize: 18)
by swift 4.1 syntax * UIView.animate * CGAffineTransform
e.g. missing commas, verb tense correction, and the like.
BaseButtonBarPagerTabStripViewController.buttonBarView.selectedBarVerticalAlignment did not respect the settings in the style. This meant that any subclasses of BaseButtonBarPagerTabStripViewController could not change the vertical alignment of the selected bar.
Prefer the indicatorInfo's accessibilityLabel
Add UIKit import as it creates a compile time error
As a better approach we may use another branch https://github.com/trading-point/XLPagerTabStrip/tree/fork_from_original for the same reasons, consisting of the exact same changes. The only difference will be in the commit structure, that is easier to follow: with the latest original |
This PR includes
Reasons
Results