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 Button font and icon sizes #2339

Merged
merged 9 commits into from
Dec 12, 2023
Merged

Fix Button font and icon sizes #2339

merged 9 commits into from
Dec 12, 2023

Conversation

connor-baer
Copy link
Member

Purpose

@borisborisov-ux noticed two design details that I had overlooked in Figma while implementing #2299.

Approach and changes

  • Reduced the size of the Button component's navigation icon.
  • Reduced the font-size of small Buttons.

Definition of done

  • Development completed
  • Reviewers assigned
  • Unit and integration tests
  • Meets minimum browser support
  • Meets accessibility requirements

@connor-baer connor-baer added the 🐞 bug Something isn't working as it should label Dec 12, 2023
@connor-baer connor-baer requested a review from a team as a code owner December 12, 2023 09:36
Copy link

changeset-bot bot commented Dec 12, 2023

🦋 Changeset detected

Latest commit: 392d63c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@sumup/circuit-ui Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

vercel bot commented Dec 12, 2023

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

Name Status Preview Comments Updated (UTC)
oss-circuit-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 12, 2023 11:55am

Copy link

codecov bot commented Dec 12, 2023

Codecov Report

Merging #2339 (392d63c) into canary (ef5ed0a) will increase coverage by 0.00%.
Report is 1 commits behind head on canary.
The diff coverage is 60.00%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           canary    #2339   +/-   ##
=======================================
  Coverage   96.97%   96.97%           
=======================================
  Files         252      252           
  Lines       20274    20275    +1     
  Branches     1263     1263           
=======================================
+ Hits        19661    19662    +1     
  Misses        600      600           
  Partials       13       13           
Files Coverage Δ
packages/circuit-ui/components/Button/shared.tsx 92.53% <60.00%> (+0.03%) ⬆️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug Something isn't working as it should 🗂 circuit-ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant