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 the correct class name in bold label example #784

Merged
merged 2 commits into from
Jun 13, 2018
Merged

Conversation

kr8n3r
Copy link

@kr8n3r kr8n3r commented Jun 12, 2018

Currently we talk about a bold label modifier class
https://govuk-frontend-review.herokuapp.com/components/label
which doesn't exist.

It's been replaced by .govuk-label--s class.

Updated to add the correct modifier class to the yaml file and README
https://govuk-frontend-review-pr-784.herokuapp.com/components/label

Fixes #783

@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-784 June 12, 2018 15:44 Inactive
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-784 June 12, 2018 15:46 Inactive
CHANGELOG.md Outdated
@@ -165,6 +165,9 @@ Note: We're not following semantic versioning yet, we are going to talk about th
and footer (the OGL logo) by marking them as non-focusable elements
([PR #774](https://github.com/alphagov/govuk-frontend/pull/774))

- Update CSS modifier class for a bold label
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think I'd know what this meant if I read this without looking at the PR.

What about something like "Use the correct class name in bold label example (govuk-label--s rather than govuk-label--bold)"

Copy link
Author

Choose a reason for hiding this comment

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

yeah, was a bit optimistic with the title. updated

@kr8n3r kr8n3r changed the title Update CSS modifier class for a bold label Use the correct class name in bold label example Jun 13, 2018
Copy link
Contributor

@36degrees 36degrees left a comment

Choose a reason for hiding this comment

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

👍

@kr8n3r kr8n3r merged commit 9fd1798 into master Jun 13, 2018
@kr8n3r kr8n3r deleted the bol-label-variant branch June 13, 2018 08:42
@hannalaakso hannalaakso mentioned this pull request Jun 14, 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.

3 participants