-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
buttonBarView #375
buttonBarView #375
Conversation
BaseButtonBarPagerTabStripViewController.swift: buttonBarView in code crash
Add "badge" and "userInfo" as extra properties?
Delete repeat init method
IndicatorInfo add full initialize
It's very useful! Hope to merge into master! 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We totally missed the error with the weak reference, good catch !
But i'd rather we have the same thing that was done in ButtonBarPagerTabStripViewController
so if you can change it i'll merge it.
Also, please remove the changes to the IndicatorInfo
that seem to be related to some custom implementation of a badge and don't forget to squash everything into one commit.
Cheers
@santiagofm Maybe we should make a discussion about |
@santiagofm |
I'm not familiar with Pull Request,what should I do? |
@codwam I think squashing those 4 commits into just one and then amending it to remove the |
Sorry, I don't know how to revert. |
BaseButtonBarPagerTabStripViewController.swift:
buttonBarView in code crash