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

Checkboxes - none of these JS behaviour #756

Merged
merged 18 commits into from
Aug 23, 2021

Conversation

andymantell
Copy link
Contributor

@andymantell andymantell commented Jul 26, 2021

Description

Add JavaScript behaviour to a set of checkboxes which unsets other checkboxes in a given group when the "None" option is checked, as in the gif below:

ezgif-2-6af8cacf2150

Also Fixes #609

Checklist

  • Tested against our testing policy (Resolution, Browser & Accessibility)

(Tested in all except Samsung internet as browserstack wasn't working for it)

@andymantell
Copy link
Contributor Author

@tomdoughty @chrimesdev I think this is ready for review now. Just about to do some browser testing but I think it's complete otherwise - happy to jump on a call and discuss if that is easiest - it might be good to highlight some places where I have had to deviate from the govuk-frontend implementation

@andymantell andymantell marked this pull request as ready for review July 27, 2021 12:47
Copy link
Contributor

@tomdoughty tomdoughty left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@chrimesdev
Copy link
Member

Cheers Andy!

@chrimesdev chrimesdev merged commit d1345ef into master Aug 23, 2021
@chrimesdev chrimesdev deleted the feature/checkboxes-none-of-these branch August 23, 2021 10:09
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.

radiobutton/checkboxed in nested fieldsets inadvertently setting conditional to hidden
3 participants