Skip to content

Update css guides#794

Merged
enatario merged 11 commits into
mainfrom
en-update-css-guides
Jun 12, 2026
Merged

Update css guides#794
enatario merged 11 commits into
mainfrom
en-update-css-guides

Conversation

@enatario

Copy link
Copy Markdown
Contributor

Our CSS guides were pretty stale and not comprehensive. This updates them to include more modern approaches, including recommending native CSS over SCSS (this removes the Sass guides). This also leverages Roux as a baseline for best practices.

@vendelal vendelal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm so grateful you took the time to make these updates! I know I will reference this in the near future knowing it's current and there were some great tips in here that were new to me/that I often forget!

Comment thread css/README.md
Comment thread css/README.md
Comment thread css/README.md
Comment thread css/README.md Outdated
Comment thread css/README.md
Comment thread css/README.md Outdated
Co-authored-by: Vendela Colavecchio (Larsson) <vendelalinnea@mac.com>

@aspencer116 aspencer116 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a great update 👏 thanks for working on this! Definitely worth promoting in a blog or social post once it's merged!

Comment thread css/README.md
Comment thread css/README.md
Comment thread css/README.md
Comment thread css/README.md Outdated
Comment thread css/README.md Outdated
@enatario enatario force-pushed the en-update-css-guides branch from c3f2e5a to 1cf6d0e Compare June 5, 2026 16:04

@pangolingo pangolingo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I love this update!
I left some suggestions, but none of them are critical.

Comment thread css/README.md Outdated
Comment thread css/README.md Outdated
Comment thread css/README.md
Comment thread css/README.md Outdated
Comment thread css/README.md Outdated
Comment thread sass/README.md
enatario and others added 2 commits June 10, 2026 13:55
Co-authored-by: Dave Iverson <iversondave@gmail.com>
@enatario enatario force-pushed the en-update-css-guides branch from c31cf69 to ced1b0b Compare June 10, 2026 20:34
@enatario enatario force-pushed the en-update-css-guides branch from ced1b0b to fea2465 Compare June 10, 2026 20:36

@brian-penguin brian-penguin left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!

@enatario enatario merged commit f171612 into main Jun 12, 2026
2 checks passed
@enatario enatario deleted the en-update-css-guides branch June 12, 2026 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants