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

Add govuk-visually-hidden-focusable class #859

Merged
merged 2 commits into from
Jul 4, 2018
Merged

Conversation

alex-ju
Copy link
Contributor

@alex-ju alex-ju commented Jul 2, 2018

This PR introduces a new class .govuk-visually-hidden-focusable and deprecates .govuk-visually-hidden-focussable

screen shot 2018-07-04 at 11 01 13

@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-859 July 2, 2018 14:33 Inactive
@alex-ju alex-ju force-pushed the fix-focusable-typo branch from bb57578 to 0aef756 Compare July 4, 2018 09:24
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-859 July 4, 2018 09:24 Inactive
@alex-ju alex-ju force-pushed the fix-focusable-typo branch from 0aef756 to ecd2af1 Compare July 4, 2018 10:02
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-859 July 4, 2018 10:02 Inactive
@alex-ju alex-ju changed the title Fix typo Add govuk-visually-hidden-focusable class Jul 4, 2018
.govuk-visually-hidden-focussable {
@include govuk-visually-hidden-focusable;
}

Copy link
Contributor

Choose a reason for hiding this comment

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

Can we do

.className,
.secondClassName {
@include ...
}

Or does that run into problems?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Tried, but Sass docs struggles with that when @ ing the deprecation

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Also I think by generally defining the deprecated code separately make it easier to remove on major releases. We're doing the same with the font baseline adjustments.

Copy link
Contributor

Choose a reason for hiding this comment

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

Sounds good

Copy link

@kr8n3r kr8n3r left a comment

Choose a reason for hiding this comment

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

thanks Alex

@alex-ju alex-ju merged commit 61d48a5 into master Jul 4, 2018
@alex-ju alex-ju deleted the fix-focusable-typo branch July 4, 2018 12:45
@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