-
Notifications
You must be signed in to change notification settings - Fork 229
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
Percent padding support. Solves #186 #187
Percent padding support. Solves #186 #187
Conversation
Hi, we're using this feature in apps with millions of users, any chance of being reviewed? Thanks! |
Sorry @albertocsouto, I have missed that PR |
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.
Thanks for this addition.
Please update the README, and also add a simple unit test for each methods in PaddingSpec
Also, please update your work branch with the latest |
c6c1566
to
bf8394b
Compare
Applied suggested changes, thanks for your patience :) |
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.
Excellent 👌
bf8394b
to
94c8458
Compare
94c8458
to
cf5e2f7
Compare
Release created https://github.com/layoutBox/FlexLayout/releases/tag/1.3.23 |
Padding by percent added in the same fashion as margins. Solves #186