Skip to content
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

fix: update Breadcrumb component to new design #373

Merged
merged 3 commits into from
Jul 31, 2024

Conversation

samanthadotdesign
Copy link
Contributor

  • Update typography and semantic colour tokens
  • Changed underline offset to not cut off at descenders
  • Changed chevron to smaller size

@samanthadotdesign samanthadotdesign requested a review from a team as a code owner July 26, 2024 06:46
Copy link

vercel bot commented Jul 26, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
isomer-studio ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 31, 2024 6:21am

Copy link

linear bot commented Jul 26, 2024

@karrui karrui changed the title Samanthalee/isom 1275 breadcrumbs styling change fix: update various styles in Callout and Breadcrumb components Jul 26, 2024
@dcshzj
Copy link
Contributor

dcshzj commented Jul 29, 2024

Might need to rebase? Chromatic did not manage to run for this

@karrui
Copy link
Contributor

karrui commented Jul 31, 2024

Taking over this PR

@karrui karrui requested review from dcshzj and seaerchin July 31, 2024 04:37
@karrui karrui changed the title fix: update various styles in Callout and Breadcrumb components fix: update Breadcrumb component to new design Jul 31, 2024
@karrui karrui force-pushed the samanthalee/isom-1275-breadcrumbs-styling-change branch from c41116d to 8a325bb Compare July 31, 2024 06:15
@karrui karrui merged commit cf37578 into main Jul 31, 2024
16 checks passed
@karrui karrui deleted the samanthalee/isom-1275-breadcrumbs-styling-change branch July 31, 2024 06:33
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.

4 participants