-
Notifications
You must be signed in to change notification settings - Fork 231
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 guidance for divider and "none of these" to Checkboxes component #1535
Add guidance for divider and "none of these" to Checkboxes component #1535
Conversation
✔️ You can preview this change here: 🔨 Explore the source changes: bb69762 🔍 Inspect the deploy log: https://app.netlify.com/sites/govuk-design-system-preview/deploys/60d461cba3803500075b003b 😎 Browse the preview: https://deploy-preview-1535--govuk-design-system-preview.netlify.app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added suggestions to guidance content ahead of going to review by the Design System working group.
@calvin-lau-sig7 one other thing, I wasn't sure where best to place the error message guidance for the None option. One the one hand, it probably makes sense for it to be alongside the other error message guidance (which is where I put it), but on the other, it does make referring to it within the section a bit clumsy. I did have "(see below)" initially, but that felt worse... 🤔 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple of copy points that came up in the working group review (so far).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Content looks good! Added a few comments.
022fe1a
to
337429c
Compare
337429c
to
ac602c3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Guidance text looks good to me! I've made one minor suggestion for better clarity.
…e Checkboxes component.
f56715f
to
bb69762
Compare
This updates the GOV.UK Design System website with guidance on the use of a "None of these" option for the Checkboxes component.
➡️ Preview of the Checkboxes component with new guidance
Uses a pre-release version of
govuk-frontend
, currently open as PR #2151.NOTE: Do not merge until the
govuk-frontend
PR has been merged and released, and then update this PR to use the released version.