Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#14795 closed enhancement (duplicate)

Twenty Ten Theme - skip link CSS fixes

Reported by: elfin's profile 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 13 years ago.

Download all attachments as: .zip

Change History (8)

#2 @elfin
13 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.

#3 @nacin
13 years ago

  • Keywords twentyten-a11y added

#4 @iammattthomas
13 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.

#5 @elfin
13 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.

#6 @elfin
13 years ago

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

#7 @nacin
13 years ago

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