Skip to content
This repository was archived by the owner on Apr 30, 2026. It is now read-only.

styling example#6

Open
laurensvanpoucke wants to merge 1 commit into
masterfrom
styling
Open

styling example#6
laurensvanpoucke wants to merge 1 commit into
masterfrom
styling

Conversation

@laurensvanpoucke
Copy link
Copy Markdown

Feedback is zeker welkom!

@doozMen
Copy link
Copy Markdown
Contributor

doozMen commented Dec 29, 2016

Best een reviewer assignen voor comments

@JasperVercammen
Copy link
Copy Markdown

@laurensvanpoucke Ziet er goed uit! Juist 1 vraagje: Moeten helpers en modules ook beginnen met h- en m-? Die layouts snap ik en vind ik duidelijk, maar m-button vind ik persoonlijk geen verbetering over button (of btn) of l-text-center over text-center...

@laurensvanpoucke
Copy link
Copy Markdown
Author

@JasperVercammen Mij lijkt het nog net iets duidelijker als de prefixes m en h erbij staan. Dan zie je direct in de markup dat het om een module/helper/... gaat. Als er geen prefixes bij staan is het css dat niet tot modules/layouts/helpers/... hoort. Die mag er trouwens ook zijn volgens mij. Meestal plaats ik deze in een directory "non-modular".

@JasperVercammen
Copy link
Copy Markdown

Alright! Kan ik me in vinden. Voor mij prima dan, duidelijk voorbeeldje :-)

@SamVanHoof
Copy link
Copy Markdown

I like it! Heel gestructureerd, niet veel op aan te merken. Initieel ook niet echt fan van de prefixes, al is het eerder omdat deze verkeerd opgevat kunnen worden, maar kan dat ik dan een beetje aan het mierenneuken ben. Bij "m-button" dacht ik bijvoorbeeld eerst aan "mobile-button", bij grotere applicaties of applicaties over meerdere platformen kan dit volgens mij wel verwarrend worden.

@SamVanHoof
Copy link
Copy Markdown

Misschien interessant om hier ook een voorbeeld van te maken met een React of Angular 2 component based structuur

@dgyesbreghs
Copy link
Copy Markdown

@JasperVercammen @SamVanHoof Is deze PR nog van toepassing?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants