Skip to content

Add Winging It 34 - #1156

Merged
SondraE merged 2 commits into
mainfrom
winging-it-34
Aug 20, 2026
Merged

Add Winging It 34#1156
SondraE merged 2 commits into
mainfrom
winging-it-34

Conversation

@SondraE

@SondraE SondraE commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Related Issue(s)

305

Steps to test/reproduce

Latest post

@SondraE
SondraE requested review from jgerigmeyer and stacyk August 11, 2026 21:11
@github-actions

Copy link
Copy Markdown

View diff of compiled files (may take a few minutes): https://github.com/oddbird/oddleventy-built/compare/main..winging-it-34

@SondraE

SondraE commented Aug 11, 2026

Copy link
Copy Markdown
Contributor Author

@stacyk I used your notes from the Show Notes document. I wasn't sure where Variable fonts and animations from the links section fit in. Hoping we can merge this by Thursday.

Comment thread content/blog/wingingit/winging-it-34.md Outdated
## Links:

- [Custom properties and grid](https://www.oddbird.net/)
- Variable fonts and animations

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This needs a link.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Miriam showed animations when we were on OddCourses, so the link should be on this text, and not on its own line below

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe we should add the word demo to the end of each bullet so it doesn't look like we are linking to an explanation of the property but instead where we played with inspecting tools?

Comment thread content/blog/wingingit/winging-it-34.md Outdated

## Links:

- [Custom properties and grid](https://www.oddbird.net/)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is this supposed to link to our homepage? If so:

Suggested change
- [Custom properties and grid](https://www.oddbird.net/)
- [Custom properties and grid](/)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

yes, we demoed OddSite for custom properties and grid

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ok, I'm fine with this, but also I'm not sure we need to link to our own site :)

@SondraE

SondraE commented Aug 12, 2026

Copy link
Copy Markdown
Contributor Author

@jgerigmeyer I copied the links from the show notes doc, but was confused what was supposed to link where. Need clarification from @stacyk.

@stacyk stacyk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@SondraE let me know if my comments make sense.

Comment thread content/blog/wingingit/winging-it-34.md Outdated

## Links:

- [Custom properties and grid](https://www.oddbird.net/)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

yes, we demoed OddSite for custom properties and grid

Comment thread content/blog/wingingit/winging-it-34.md Outdated
## Links:

- [Custom properties and grid](https://www.oddbird.net/)
- Variable fonts and animations

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Miriam showed animations when we were on OddCourses, so the link should be on this text, and not on its own line below

Comment thread content/blog/wingingit/winging-it-34.md
Comment thread content/blog/wingingit/winging-it-34.md Outdated
## Links:

- [Custom properties and grid](https://www.oddbird.net/)
- Variable fonts and animations

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe we should add the word demo to the end of each bullet so it doesn't look like we are linking to an explanation of the property but instead where we played with inspecting tools?

@SondraE
SondraE requested a review from jgerigmeyer August 19, 2026 18:33
@SondraE

SondraE commented Aug 19, 2026

Copy link
Copy Markdown
Contributor Author

@jgerigmeyer @stacyk Does this revision help clarify the links?

@SondraE
SondraE requested a review from stacyk August 19, 2026 18:33

@stacyk stacyk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Works for me!

@SondraE
SondraE merged commit e17b8ea into main Aug 20, 2026
10 checks passed
@SondraE
SondraE deleted the winging-it-34 branch August 20, 2026 15:53
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.

3 participants