Skip to content

Commit

Permalink
Cleanup/fix after the sr-only to visually-hidden renaming (twbs#31359)
Browse files Browse the repository at this point in the history
* sr-only -> visually-hidden cleanup

It seems the old screen-readers.md file was left behind, and forgot to rename the mixin

* Fix broken mixins for visually-hidden
  • Loading branch information
patrickhlauke authored and olsza committed Oct 3, 2020
1 parent a76f022 commit 3871dc4
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 29 deletions.
2 changes: 1 addition & 1 deletion scss/_mixins.scss
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
@import "mixins/breakpoints";
@import "mixins/image";
@import "mixins/resize";
@import "mixins/screen-reader";
@import "mixins/visually-hidden";
@import "mixins/reset-text";
@import "mixins/text-truncate";

Expand Down
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
// stylelint-disable declaration-no-important

// Only display content to screen readers
// Hide content visually while keeping it accessible to assistive technologies
//
// See: https://a11yproject.com/posts/how-to-hide-content/
// See: https://hugogiraudel.com/2016/10/13/css-hide-and-seek/

@mixin visually-hidden {
@mixin visually-hidden() {
position: absolute !important;
width: 1px !important;
height: 1px !important;
Expand All @@ -21,7 +21,7 @@
//
// Useful for "Skip to main content" links; see https://www.w3.org/TR/2013/NOTE-WCAG20-TECHS-20130905/G1

@mixin visually-hidden-focusable {
@mixin visually-hidden-focusable() {
&:not(:focus) {
@include visually-hidden();
}
Expand Down
25 changes: 0 additions & 25 deletions site/content/docs/5.0/helpers/screen-readers.md

This file was deleted.

0 comments on commit 3871dc4

Please sign in to comment.