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

Use GOV.UK elements v3.0.1 #364

Merged
merged 7 commits into from
Mar 21, 2017
Merged

Use GOV.UK elements v3.0.1 #364

merged 7 commits into from
Mar 21, 2017

Conversation

gemmaleigh
Copy link
Contributor

@gemmaleigh gemmaleigh commented Mar 21, 2017

This PR updates the examples for radio buttons and checkboxes to match those used by GOV.UK elements.

It includes breaking changes - as the markup for custom radio buttons and checkboxes has changed.

In summary:

  • use govuk-elements-sass v3.0.1
  • remove deprecated selection-buttons.js - the radios and checkboxes no longer require JavaScript
  • update show-hide-content.js (to toggle hidden content, using the data-target attribute)
  • update examples in docs using radio buttons and checkboxes
  • replace block-label with multiple-choice in comments

Screenshots:

example radio buttons and checkboxes

Note: There are also breaking changes to the classnames used for errors and validation, but the prototype kit does not have examples of these.

This is no longer required with the radios and checkboxes released in
govuk-elements-sass v3.0.0.
To use the new markup released in GOV.UK elements 3.0.0.
Update examples of radio buttons and checkboxes.

Replace ‘.block-label` with `.multiple-choice` as the block-label class
no longer exists.
Increase the spacing between the radio button and checkbox examples.
This had nested form elements, which was a mistake. Removing these.
@joelanman joelanman merged commit a21f35a into master Mar 21, 2017
@joelanman joelanman deleted the use-elements-3.0.1 branch March 21, 2017 17:10
@joelanman joelanman mentioned this pull request Apr 5, 2017
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.

2 participants