Twenty Eleven broken screen reader link

[gluten]

The Twenty Eleven theme adds two skip links for accessibility: "skip to primary content" and "skip to secondary content". The problem is, the "secondary content" does not always exist. All singular posts and pages do not have the linked-to secondary content section, rendering the link broken on all of those pages.

Since none of the default themes since Twenty Eleven have had a second skip link, it seems like removing the secondary skip link entirely would be the best course of action.

[bcworkz]

Removing the link would be easy, but I don't like the idea of removing established functionality. It's not so difficult to make the link output conditional on !is_singular().

[johnbillion]

The title attribute on these links are also redundant.

[bcworkz]

Title attributes removed

[bcworkz]

Replying to johnbillion:

The title attribute on these links are also redundant.

Nearly all title attributes are at least conceptually redundant. These are literally redundant! As it happens the original 26950.patch is unusable, I somehow dropped the ! modifier to is_singular(), so 26950.no-title.patch is the only usable patch at this point.

[lancewillett]

In 27569:

Twenty Eleven: only show secondary skip link on pages with secondary content, and remove duplicate title attribute in site title output. Props bcworkz, fixes #26950.