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

Add post on improving accessibility of Rails forms #153

Merged
merged 13 commits into from
Jan 5, 2025

Conversation

peteryates
Copy link
Member

No description provided.

@peteryates peteryates force-pushed the add-finishing-touches-for-forms-post branch from fc7cc3b to 185fdaf Compare December 21, 2024 23:36

This causes a few problems.

The GOV.UK design system includes some complex rules around linking to fields from the error summary. For example, errors on checkbox or radio button fields should link to the first checkbox or radio button.
Copy link
Collaborator

Choose a reason for hiding this comment

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

This bit isn’t explained in the post. Worth adding a separate section to explain link_errors: true?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, this post needs three things; two seems too few!

Copy link
Collaborator

Choose a reason for hiding this comment

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

Header image too! 😀

@paulrobertlloyd
Copy link
Contributor

paulrobertlloyd commented Dec 22, 2024

I've got a great idea for an illustration… will rustle something up in a bit!


The [GOV.UK formbuilder](https://govuk-form-builder.netlify.app) makes it easy for Ruby on Rails developers to build forms that follow the rules set out in the [GOV.UK design system](https://design-system.service.gov.uk/).

It takes care of the markup, provides a nice API for customising the fields to meet the needs of your service and takes care of clearly dispalying any error messages.
Copy link
Member Author

Choose a reason for hiding this comment

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

Typo in 'displaying'

Copy link
Collaborator

@frankieroberto frankieroberto left a comment

Choose a reason for hiding this comment

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

Some minor suggestions.

app/posts/finishing-touches-for-forms.md Outdated Show resolved Hide resolved
app/posts/finishing-touches-for-forms.md Outdated Show resolved Hide resolved
@paulrobertlloyd
Copy link
Contributor

@peteryates Added an illustration – home you like it!? Also simplified the side-by-side example image, making it less busy and a smaller file size.

@paulrobertlloyd paulrobertlloyd changed the title Add post on improving accessibilty of Rails forms Add post on improving accessibility of Rails forms Dec 22, 2024
@peteryates
Copy link
Member Author

I love the graphic at the top 🤩

Thanks for the feedback, I think I've addressed it and added the third section on link_errors.

app/posts/finishing-touches-for-forms.md Outdated Show resolved Hide resolved
app/posts/finishing-touches-for-forms.md Show resolved Hide resolved
app/posts/finishing-touches-for-forms.md Show resolved Hide resolved
app/posts/finishing-touches-for-forms.md Show resolved Hide resolved
Copy link
Collaborator

@frankieroberto frankieroberto left a comment

Choose a reason for hiding this comment

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

One minor (but crucial) typo. 😄

app/posts/finishing-touches-for-forms.md Outdated Show resolved Hide resolved
app/posts/finishing-touches-for-forms.md Outdated Show resolved Hide resolved
app/posts/finishing-touches-for-forms.md Show resolved Hide resolved
Copy link
Collaborator

@frankieroberto frankieroberto left a comment

Choose a reason for hiding this comment

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

Looking good. ✅

@paulrobertlloyd paulrobertlloyd self-requested a review December 27, 2024 21:34
@peteryates peteryates force-pushed the add-finishing-touches-for-forms-post branch from 5923477 to 61c7291 Compare January 4, 2025 21:00
@peteryates peteryates merged commit f6ba49b into main Jan 5, 2025
2 checks passed
@peteryates peteryates deleted the add-finishing-touches-for-forms-post branch January 5, 2025 12:24
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.

3 participants