Skip to content

Conversation

@jorytindall
Copy link
Contributor

@jorytindall jorytindall commented May 5, 2025

📌 Summary

In testing some ways to extend the Dropdown to support Advanced Table features it was discovered that the checkbox and radio list items used a different font weight from the rest of the list items.

This PR:

  • Updates the text to use Hds::Text::Body similarly to the other list item components
  • Changes the font weight to medium
  • Updates the count to use Hds::Text::Body in radio and checkbox to match the implementation in the checkmark.

📸 Screenshots

Checkbox

Before:
Screenshot 2025-05-05 at 1 15 55 PM

After:
Screenshot 2025-05-05 at 1 34 50 PM

Radio

Before:
Screenshot 2025-05-05 at 1 16 32 PM

After:
Screenshot 2025-05-05 at 1 16 53 PM

🔗 External links

Jira ticket: HDS-4837
Figma file: Branch


👀 Component checklist

💬 Please consider using conventional comments when reviewing this PR.

@vercel
Copy link

vercel bot commented May 5, 2025

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

Name Status Preview Updated (UTC)
hds-showcase ✅ Ready (Inspect) Visit Preview May 5, 2025 8:21pm
hds-website ✅ Ready (Inspect) Visit Preview May 5, 2025 8:21pm

Copy link
Contributor

@didoo didoo left a comment

Choose a reason for hiding this comment

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

👍

@alex-ju alex-ju added this to the [email protected] milestone May 6, 2025
Copy link
Contributor

@majedelass majedelass left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@jorytindall jorytindall merged commit 1b40adb into main May 6, 2025
16 checks passed
@jorytindall jorytindall deleted the jorytindall/hds-4837/dropdown-font-weight branch May 6, 2025 19:21
@hashibot-hds hashibot-hds mentioned this pull request May 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants