Skip to content

Conversation

@lindsaym-fa
Copy link
Contributor

@lindsaym-fa lindsaym-fa commented Oct 3, 2025

Relies on https://github.com/shoelace-style/webawesome-pro/pull/58

  • Adds a link to new Figma Design Kit doc to the sidebar
  • Adds common styles for step icons to outline ordered instructions

@talbs, mind letting me know your thoughts on:

  1. Where the Figma kit belongs in the sidebar
  2. Whether or not the .step-icon styles belong in utils.css?

I figure .step-icon would be good to use in the 2FA flow too, which I can make a quick PR to update if everything's kosher.

@vercel
Copy link

vercel bot commented Oct 3, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
webawesome Ready Ready Preview Oct 16, 2025 7:19pm

@lindsaym-fa lindsaym-fa requested a review from talbs October 3, 2025 20:23
Copy link
Contributor

@talbs talbs left a comment

Choose a reason for hiding this comment

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

Nice work adding this.

I left some thoughts on both where to link and the step-icon utility inline. Let me know what you think on each and if you want to chat anything out.

Copy link
Contributor

@talbs talbs left a comment

Choose a reason for hiding this comment

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

Great success! Get on the road and 🚚 this to Mergeburgh!

}

/* step icons for ordered instructions */
.step-icon {
Copy link
Contributor

Choose a reason for hiding this comment

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

📶 💪

}
}

/* Images & Figures */
Copy link
Contributor

Choose a reason for hiding this comment

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

💥

<strong>Need a hand?</strong>
<wa-button size="small" appearance="filled" variant="neutral" href="https://github.com/shoelace-style/webawesome/issues" target="_blank">
<wa-icon slot="prefix" name="bug"></wa-icon>
<wa-icon slot="start" variant="regular" name="bug"></wa-icon>
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice fix!

</wa-button>
<wa-button size="small" appearance="filled" variant="neutral" href="https://github.com/shoelace-style/webawesome/discussions" target="_blank">
<wa-icon slot="prefix" name="message-question"></wa-icon>
<wa-icon slot="start" variant="regular" name="message-question"></wa-icon>
Copy link
Contributor

Choose a reason for hiding this comment

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

Equally nice fix!

@lindsaym-fa lindsaym-fa merged commit d792ac0 into next Oct 16, 2025
2 checks passed
@lindsaym-fa lindsaym-fa mentioned this pull request Oct 28, 2025
38 tasks
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