Attachment 123146 Details for Bug 28469 – Bug 28469: Move "Skip to main content" to top of OPAC page

[patch] Bug 28469: Move "Skip to main content" to top of OPAC page

Bug-28469-Move-Skip-to-main-content-to-top-of-OPAC.patch (text/plain), 2.08 KB, created by Nick Clemens (kidclamp) on 2021-07-23 16:38:18 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2021-07-23 16:38:18 UTC

Size: 2.08 KB

patch

obsolete

>From cabeec02e569f50ba7729a91cdaf470a70ecc941 Mon Sep 17 00:00:00 2001
>From: David Cook <dcook@prosentient.com.au>
>Date: Thu, 27 May 2021 03:34:30 +0000
>Subject: [PATCH] Bug 28469: Move "Skip to main content" to top of OPAC page
>
>This patch moves the "Skip to main content" button out of the nav
>element and to the top of the <body> element, so that it's the first
>(potentially) viewable element on the page.
>
>This means you can inject content above the navbar, which has
>a relative position, and it won't disrupt the display of the
>"Skip to main content" button.
>
>To test:
>1) Apply patch
>2) Same test plan as Bug 22807:
>Go to any page in the OPAC and hit the tab key. The "Skip to main
>content" link should appear. Tabbing away from it should cause the link
>to hide. Hitting the ENTER key when the link is highlighted should cause
>the page to scroll to the main content.
>
>Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
>Signed-off-by: Wainui Witika-Park <wainuiwitikapark@catalyst.net.nz>
>
>Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
>---
> koha-tmpl/opac-tmpl/bootstrap/en/includes/masthead.inc | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/includes/masthead.inc b/koha-tmpl/opac-tmpl/bootstrap/en/includes/masthead.inc
>index e44ae2f111..49a875d304 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/includes/masthead.inc
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/includes/masthead.inc
>@@ -9,10 +9,10 @@
> [% SET OpacCustomSearch = KohaNews.get( location => "OpacCustomSearch", lang => lang, library => branchcode, blocktitle => 0 ) %]
> [% SET OpacLoginInstructions = KohaNews.get( location => "OpacLoginInstructions", lang => lang, library => branchcode ) %]
> 
>+<button id="scrolltocontent">Skip to main content</button>
> <div id="wrapper">
>     <div id="header-region" class="noprint">
>         <nav class="navbar navbar-expand">
>-            <button id="scrolltocontent">Skip to main content</button>
>             <div id="logo">
>             <a class="navbar-brand" href="/cgi-bin/koha/opac-main.pl">
>                 [% IF ( LibraryNameTitle ) %]
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 28469: 121442 | 121464 | 121468 | 123146