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

docs: form and button guidance updates #3722

Merged
merged 8 commits into from
Sep 27, 2023

Conversation

aagonzales
Copy link
Member

@aagonzales aagonzales commented Aug 29, 2023

Closes #3546
Closes #3578

Changelog

Note, some of the images aren't updating in the preview so just check them in the "file changed" tab. This doesn't include all the content enhancement we wanted to do but it at least corrects the obviously wrong rules.

Changed

/patterns/forms-pattern/#buttons

  • Updated the button emphasis section text to match what we say on the button usage page about primary button placement.
  • Updated the button emphasis section with new images
  • Updated all full-form images to include instructional text about required fields
  • Margin vs full bleed: updated to say primary is always on the bottom

/components/form

  • Updated images with the correct primary/secondary ordering
  • Updated images to include instructional text.

@vercel
Copy link

vercel bot commented Aug 29, 2023

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

Name Status Preview Comments Updated (UTC)
carbondesignsystem ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 27, 2023 11:07pm

@laurenmrice
Copy link
Member

@thyhmdo or @jeanservaas, when you have some time this sprint can you review this PR? Thank you!

Copy link
Member

@thyhmdo thyhmdo left a comment

Choose a reason for hiding this comment

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

Looks great!

@kodiakhq kodiakhq bot merged commit 28a43d3 into carbon-design-system:main Sep 27, 2023
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants