WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#14795 closed enhancement (duplicate)

Twenty Ten Theme - skip link CSS fixes

Reported by: elfin Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Accessibility Keywords: twentyten-a11y
Focuses: Cc:

Description

Having a skip link is good, not showing it when tabbing through the page is bad. These minor tweaks fix this and display it 'over' the header image.

Attachments (1)

screen-reader-text.patch (588 bytes) - added by elfin 4 years ago.

Download all attachments as: .zip

Change History (8)

elfin4 years ago

comment:2 elfin4 years ago

Oh and ideally it shouldn't be called screen-reader-text as it is inaccurate. Sighted people who navigate using a keyboard find it useful as well.

comment:3 nacin4 years ago

  • Keywords twentyten-a11y added

comment:4 iammattthomas4 years ago

I like the idea here, but the styling isn't great -- no background, white text, full-width of the page, makes the link difficult to read, depending on the header image. Should be Helvetica to match the text in the menu, too.

comment:5 elfin4 years ago

can you confirm which browser you used? the skip link background was supposed to be restricted to the same width as the the horizontal menu.

comment:6 elfin4 years ago

I won't correct this one as #14798 supercedes it.

comment:7 nacin4 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to duplicate
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.