-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
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
[Container] Document the classes API #15919
[Container] Document the classes API #15919
Conversation
No bundle size changes comparing 831c6cc...d2ee5ac |
@divyanshutomar I'm sorry, I don't understand how #15919 is different to #15872. They are both breaking changes. I'm doing a bit of changes. I have labeled #15872 for v5 (we can probably merge it around Q1 2020). Now, let's keep this one for adding the missing documentation :). I'm updating it. |
30160da
to
d2ee5ac
Compare
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.
Let me know what you think of the changes. I believe it's a first good step.
@oliviertassinari Sorry, I guess I misunderstood the breaking change. Anyway, this looks good to go! I will update the previous PR once this is merged. |
@divyanshutomar Thanks 👍 It’s always great to see documentation improvements. |
@divyanshutomar The breaking change is for people who use |
As per the discussion in #15872 , this PR forms the CSS only changes involving moving out of padding styles from
root
class to separate classgutters