primer / brand

React components and Primitives for GitHub marketing websites
https://primer.style/brand
MIT License
74 stars 33 forks source link

FormControl a11y issue remediation #779

Closed joshfarrant closed 1 month ago

joshfarrant commented 1 month ago

Summary

List of notable changes:

What should reviewers focus on?

Steps to test:

Supporting resources (related issues, external links, etc):

Contributor checklist:

Reviewer checklist:

Screenshots:

Feature Before After
FormControl.Hint associated with form field ![Before](https://github.com/user-attachments/assets/4f785c6a-3c3d-48d4-955c-94d1da5c2c12) ![After](https://github.com/user-attachments/assets/3d46862a-bdd5-4686-9d00-527443439c97)
changeset-bot[bot] commented 1 month ago

🦋 Changeset detected

Latest commit: df03fb2df02c6c591adc0ac78cdd10427a66a6f3

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

This PR includes changesets to release 6 packages | Name | Type | | ------------------------ | ----- | | @primer/brand-e2e | Minor | | @primer/react-brand | Minor | | @primer/brand-primitives | Minor | | @primer/brand-fonts | Minor | | @primer/brand-config | Minor | | @primer/brand-storybook | Minor |

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

github-actions[bot] commented 1 month ago

🟢 No design token changes found

github-actions[bot] commented 1 month ago

⚠️ Visual differences found

Our visual comparison tests found UI differences. Please review the differences by using the test artifacts to ensure that the changes were intentional. Artifacts can be downloaded and reviewed locally. Download links are available at the bottom of the workflow summary screen. ##### Example: ![artifacts section of workflow run](https://user-images.githubusercontent.com/13340707/181026915-2bda8a90-58e3-40ef-a2f6-c9c4af6e9c4a.png) If the changes are expected, please run `npm run test:visual:update-snapshots` to replace the previous fixtures.

Review visual differences