Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The "Jump to Navigation" link is too important a feature for screen reader users to have break just because we want to use the Primary Links block instead of Zen's normal primary links display. This patch makes it so that, if the normal primary links display seems to be disabled, we try gamely to see if the primary links block will display somewhere, and if so, have "Jump to Navigation" link to that instead. There might be a more elegant way to do it, though…
Comment | File | Size | Author |
---|---|---|---|
#4 | 812480-alter-jump-to-nav-link.diff | 2.64 KB | Garrett Albright |
jump-to-nav-primary-links-block.patch | 2.12 KB | Garrett Albright |
Comments
Comment #1
Garrett Albright CreditAttribution: Garrett Albright commentedComment #2
Garrett Albright CreditAttribution: Garrett Albright commentedFor the record, this patch is currently in production at http://oldtowncoffeeeureka.com/ .
Comment #3
JohnAlbinGarrett, this is a great idea, but the code necessary to implement it is quite heavy. Also, if the site admin, like me, uses menu_block instead of menu.module blocks, then the skip link breaks again.
The way I handle this is to override page.tpl and change the URL to match what is actually configured on the site.
Maybe instead we could have a theme setting to set the link?
Comment #4
Garrett Albright CreditAttribution: Garrett Albright commentedHmm, like this?
Comment #5
JohnAlbinYes, very nearly! Excellent starting point.
I tweaked it a little bit and committed it. http://drupalcode.org/project/zen.git/commit/ffb5438
Comment #6
Garrett Albright CreditAttribution: Garrett Albright commentedNitpick: It looks like in that diff, you're switching between curly and straight quotes in the theme-settings.php bit.