-
Notifications
You must be signed in to change notification settings - Fork 175
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
Layout components should have a prefix #6
Comments
ah, yes.. been mulling about this as well. |
care to share any other experiences with layout components? |
it is a little bit tricky because you don't wanna end up with a grid system. |
perhaps you could split words with double dash ( |
To my opinion, that is already putting to much assumption on what a |
So if it's just applying styling do the children, you'd go with I don't really see anything wrong with including layout within the component class like shown currently. So long as the layout styles for children are isolated from any additional styling. You might abstract the clearfix and any resets like for If you go as suggested above with |
Layout components should explicitly start with a prefix (like on SMACSS).
Areas may also be defined as well.
Apart from separating concerns, a nice feature would be "enabling" outline on areas of a layout. With that, would be easier to visualize and separate your components when creating a component library.
So this outline you have manually created could actually be automatically created based on
[class^=l-]
The text was updated successfully, but these errors were encountered: