WordPress.org
Get WordPress

Make WordPress Core

Opened 15 years ago

Closed 15 years ago

Last modified 15 years ago

#9791 closed defect (bug) (fixed)

Use a more semantic class name for screen reader class

Reported by: filosofo's profile filosofo Owned by:
Milestone: 2.8 Priority: normal
Severity: normal Version: 2.8
Component: Accessibility Keywords: has-patch screen-reader-text
Focuses: Cc:

Description

To be semantic, CSS classes (and ids) should describe meaning or function, not style.

So attached patch exchanges "invisible" for "screen-reader-text".

Attachments (1)

screen_reader_text.9791.diff​ (47.6 KB) - added by filosofo 15 years ago.

Download all attachments as: .zip

Change History (5)

@filosofo
15 years ago

#2 @markjaquith
15 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [11312]) Use a semantic class name for text targeted to screen readers. props filosofo. fixes #9791

#3 @nbachiyski
15 years ago

Isn't it targeted for browsers with no/disabled JavaScript, not for text readers specifically? This class name is really long, given the number of times it is used.

#4 @filosofo
15 years ago

I wasn't pushing for that name in particular, just something semantic.

Note: See TracTickets for help on using tickets.