-
Notifications
You must be signed in to change notification settings - Fork 8
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
Border in box-model #16
Comments
I thought about that before but these changes would separate Moving Although it's named as box-model, I'm actually thinking them as general sizing properties. Border also effects the element size like you mentioned but most of the time we do not use it that way. So, maybe naming is at fault? We can simply rename "box-model" group to "sizing"? It would really appreciated if you have any thoughts about this @wimhendrikx. |
I don't think |
All else seems to make sense to me, just border stood out as in the wrong place, so I'd vote for this one too. |
Do you guys think border should be between margin and padding or after or before margin/padding pair? Between margin and padding:
After margin and padding:
|
I would vote for "after margin" |
Border affects element sizes. Should it be in box-model? At least having option to choice will be nice, like in abandoned rational-order package.
The text was updated successfully, but these errors were encountered: