{"action":"created","issue":{"url":"https://api.github.com/repos/Otto42/theme-check/issues/42","labels_url":"https://api.github.com/repos/Otto42/theme-check/issues/42/labels{/name}","comments_url":"https://api.github.com/repos/Otto42/theme-check/issues/42/comments","events_url":"https://api.github.com/repos/Otto42/theme-check/issues/42/events","html_url":"https://github.com/Otto42/theme-check/pull/42","id":55095844,"number":42,"title":"Add a check for `.screen-reader-text`.","user":{"login":"obenland","id":1398304,"avatar_url":"https://avatars.githubusercontent.com/u/1398304?v=3","gravatar_id":"","url":"https://api.github.com/users/obenland","html_url":"https://github.com/obenland","followers_url":"https://api.github.com/users/obenland/followers","following_url":"https://api.github.com/users/obenland/following{/other_user}","gists_url":"https://api.github.com/users/obenland/gists{/gist_id}","starred_url":"https://api.github.com/users/obenland/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/obenland/subscriptions","organizations_url":"https://api.github.com/users/obenland/orgs","repos_url":"https://api.github.com/users/obenland/repos","events_url":"https://api.github.com/users/obenland/events{/privacy}","received_events_url":"https://api.github.com/users/obenland/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2015-01-21T23:59:19Z","updated_at":"2015-03-31T18:26:37Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Otto42/theme-check/pulls/42","html_url":"https://github.com/Otto42/theme-check/pull/42","diff_url":"https://github.com/Otto42/theme-check/pull/42.diff","patch_url":"https://github.com/Otto42/theme-check/pull/42.patch"},"body":"Six years ago, `.screen-reader-text` was introduced as a canonical class name for text targeted to screen readers for all of core.\r\nUnknown to many, this includes the front-end by virtue of its use in `get_search_form()`.\r\n\r\nTo ensure forward compatibility with future a11y improvements, and be able to deal with child themes adding theme support for HTML5 features which rely on that class heavily, all themes should have it defined to deal with assistive text.\r\n\r\nSee https://core.trac.wordpress.org/ticket/29699."},"comment":{"url":"https://api.github.com/repos/Otto42/theme-check/issues/comments/88198288","html_url":"https://github.com/Otto42/theme-check/pull/42#issuecomment-88198288","issue_url":"https://api.github.com/repos/Otto42/theme-check/issues/42","id":88198288,"user":{"login":"joedolson","id":1686451,"avatar_url":"https://avatars.githubusercontent.com/u/1686451?v=3","gravatar_id":"","url":"https://api.github.com/users/joedolson","html_url":"https://github.com/joedolson","followers_url":"https://api.github.com/users/joedolson/followers","following_url":"https://api.github.com/users/joedolson/following{/other_user}","gists_url":"https://api.github.com/users/joedolson/gists{/gist_id}","starred_url":"https://api.github.com/users/joedolson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/joedolson/subscriptions","organizations_url":"https://api.github.com/users/joedolson/orgs","repos_url":"https://api.github.com/users/joedolson/repos","events_url":"https://api.github.com/users/joedolson/events{/privacy}","received_events_url":"https://api.github.com/users/joedolson/received_events","type":"User","site_admin":false},"created_at":"2015-03-31T18:26:37Z","updated_at":"2015-03-31T18:26:37Z","body":"I don't think there's any major risks to this. Theme authors need to learn what this is and how to do it, and it's really extremely easy to do. If they fail to do it, it won't have any significant impact in the vast majority of cases. \r\n\r\nWhile core wants to promote the use of the class aggressively, it's not so much because we intend to start adding it all over the HTML output; it's because we want to have the option to add it in front-end HTML in the future without needing to worry about backwards compatibility every time. \r\n\r\nRight now, themes that don't define the class will have additional visible text in place where the screen reader text class is used. "}}