Attachment 130032 Details for Bug 29066 – Bug 29066: (follow-up) add aria-label and aria-hidden on icons

[patch] Bug 29066: (follow-up) add aria-label and aria-hidden on icons

Bug-29066-follow-up-add-aria-label-and-aria-hidden.patch (text/plain), 2.14 KB, created by Fridolin Somers on 2022-02-01 00:39:51 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Fridolin Somers

Created: 2022-02-01 00:39:51 UTC

Size: 2.14 KB

patch

obsolete

>From 5706b367cdf0e52eedb8b1d4ca95efa428f7bd3e Mon Sep 17 00:00:00 2001
>From: Fridolin Somers <fridolin.somers@biblibre.com>
>Date: Fri, 14 Jan 2022 20:53:54 -1000
>Subject: [PATCH] Bug 29066: (follow-up) add aria-label and aria-hidden on
> icons
>
>---
> koha-tmpl/opac-tmpl/bootstrap/en/includes/masthead.inc | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/includes/masthead.inc b/koha-tmpl/opac-tmpl/bootstrap/en/includes/masthead.inc
>index 1794184e51..8114e1e19c 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/includes/masthead.inc
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/includes/masthead.inc
>@@ -166,7 +166,7 @@
>                                                 [% UNLESS ( Koha.Preference( 'OpacAddMastheadLibraryPulldown' ) == 1 ) %]
>                                                     [% IF ( mylibraryfirst ) %]
>                                                         <span title="Search in [% Branches.GetName( mylibraryfirst ) | html %] only">
>-                                                            <i class="fa fa-question-circle-o"></i>
>+                                                            <i class="fa fa-question-circle-o" aria-hidden="true"></i>
>                                                         </span>
>                                                     [% END %]
>                                                 [% END %]
>@@ -278,8 +278,8 @@
>                                             [% END # / OpacAddMastheadLibraryPulldown %]
> 
>                                             <div class="order-5 col-sm-auto">
>-                                                <button type="submit" id="searchsubmit" class="btn btn-primary" title="Search">
>-                                                    <i class="fa fa-search"></i>
>+                                                <button type="submit" id="searchsubmit" class="btn btn-primary" title="Search" aria-label="Search">
>+                                                    <i class="fa fa-search" aria-hidden="true"></i>
>                                                 </button>
>                                             </div>
> 
>-- 
>2.34.0

Actions: View | Diff | Splinter Review

Attachments on bug 29066: 125084 | 125180 | 125181 | 128927 | 129519 | 130031 | 130032 | 130114 | 130115 | 132375 | 132376 | 132377 | 132854 | 132855 | 132947