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

Remove bottom margin on checkbox and radio labels #883

Merged
merged 2 commits into from
Jul 11, 2018

Conversation

kr8n3r
Copy link

@kr8n3r kr8n3r commented Jul 10, 2018

With the labels, legends and headings work margin-bottom: 5px was added to govuk-label.
This was intended for labels above text inputs but has been applied to checkbox and radios labels too.

So govuk-checkboxes__label and govuk-radios__label need to have margin-bottom: 0; applied to override this.

https://trello.com/c/aplfBBBF/1197-checkbox-and-radio-labels-should-not-have-a-bottom-margin

Fixes: #881

With the labels, legends and headings work
margin-bottom: 5px was added to govuk-label.

This was intended for labels above text inputs
but has been applied to checkbox and radios too.

So govuk-checkboxes__label and govuk-radios__label
need to have margin-bottom: 0; applied to
override this.
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-883 July 10, 2018 21:33 Inactive
@dashouse
Copy link

Looks good to me 👍

@36degrees 36degrees added this to the v1.1.0 milestone Jul 11, 2018
@kr8n3r kr8n3r merged commit 139649a into master Jul 11, 2018
@kr8n3r kr8n3r deleted the fix-label-margin-in-radios-and-checkboxes branch July 11, 2018 13:51
@NickColley NickColley mentioned this pull request Jul 13, 2018
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.

4 participants