search

nodeshift / web-application-reference

WIP repo to hold Red Hat/IBM web application reference
Apache License 2.0
11 stars 6 forks source link

feat: adding styles guidance #6

Closed lholmquist closed 4 months ago

ctcpip commented 6 months ago

consider adding content about css methodologies, e.g. BEM

adigidh commented 5 months ago

Not wanting this to be added in on the docs here, but wanted to start a discussion on whether it makes sense to add in a note about CSS-in-JS.

CSS-in-JS was another pattern that is or was heavily used across projects at certain point in time. styled-components and Emotion are some of the popular CSS-in-JS libraries in the React community.

lholmquist commented 5 months ago

Not wanting this to be added in on the docs here, but wanted to start a discussion on whether it makes sense to add in a note about CSS-in-JS.

If it is a pattern then i don't see why not

lholmquist commented 5 months ago

@adimaniacal thanks for the spelling updates. this just proves how bad of a speller i am 🤣

lholmquist commented 5 months ago

@adimaniacal i just added a small section related to the css-in-js pattern. mostly copy/paste of what you wrote :)

lholmquist commented 5 months ago

@paceaux i made some updates based on our latest meeting, mind taking a look

lholmquist commented 5 months ago

@paceaux sorry for the second notification, but apparently in my above comment i had a space in between the @ symbol and the p, so i don't think you would have gotten the first notification 🤷