Rez
Rez
@stamat just doing some backlog cleanup. Can we close this issue now with the release of SubNav v2?
Thanks for tidying up the InlineLink component @stamat ✨ 🧹 Before we merge this, a few things to add... - We'll need a changeset - We should remove the base.json...
Closing out as this is now stale, and corresponding work is paused.
Closing as the fix has shipped in v0.41.0
The [proposed changes](https://github.com/github/primer/issues/4936) (prev. comment☝) to `canvas-default/subtle` are paused at this time. This issue was also discussed briefly during office hours last week, specifically: - `text-subtle` is not approved for...
Thanks for the feedback @simmonsjenna ✨ The surface area of this work was initially scoped to `Text` and `Heading` compoennts, and not overrides in the React library, but in hindsight...
> @rezrah I took a look through the instances — here is my recommendation: Thanks @simmonsjenna. Requested changes have been applied: - Testimonial now uses `100` instead of `200`. -...
👋 Checking in to see how far we want to go with this change? We have many links on the site. Presumably we are [only targeting this specific compoinent](https://stunning-chainsaw-j82glqz.pages.github.io/?path=/story/components-link--default) and...
Re-opening due to: - Pending Figma updates (cc. @danielguillan) - Clarification on scope [here](https://github.com/primer/brand/issues/770#issuecomment-2456904792) (cc. @jesussandreas @simmonsjenna)
Update: Figma changes have now shipped. No additional scope added.