-
-
Notifications
You must be signed in to change notification settings - Fork 3
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
Blog post on un-mixing a mixin #614
Conversation
View diff of compiled files (may take a few minutes): https://github.com/oddbird/oddleventy-built/compare/main..un-mixing |
Co-authored-by: david herron <41588129+dvdherron@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs an image, but otherwise LGTM 👍
Co-authored-by: Jonny Gerig Meyer <jonny@oddbird.net>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should Roma Komarov should be Roman throughout the article? That's what's on their website when you click through, anyhow.
I don't think "off-switch" needs a hyphen?
You end the article with questions. You could add something like: come discuss this with me at the upcoming Cascading Layout Workshop on June 24. You can register below.
Additionally you could put the registration form CTA near the beginning of the article with an invitation something like: If you have questions about all this, come ask them. I'll be doing live Q&A at the Cascade Layouts Workshop June 24-26.
We could. He uses Roma on Mastodon - just the short form of the name. I think either is fine if we're consistent?
Does it matter that it's a layout workshop, and this post isn't really layout related at all? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great article. Great examples in it as well. Approving but I don't think it has a hero image yet?
) }} | ||
|
||
|
||
## It's hard to un-mix a previously mixed-in mixin once mixed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great heading
Co-authored-by: Stacy Kvernmo <stacyk@users.noreply.github.com>
* main: Update content/blog/elsewhere/interview/podcast/mechanical-ink.md Apply suggestions from code review Add Mechanical Ink podcast Bump the npm-minor-upgrades group with 5 updates Bump @rollup/plugin-commonjs in the npm-major-upgrades group Automated webmentions update
@SondraE posted to masto: |
@mirisuzanne @stacyk @dvdherron |
Description
Something I was thinking about after CSS Day
https://deploy-preview-614--oddleventy.netlify.app/2024/06/11/removing-mixins/
Related Issue(s)
Should I make more mention of CSS Day, or my role there, or the workshop (somehow?)