Bugzilla – Attachment 121750 Details for
Bug 28243
Accessibility: Staff interface - add captions to tables
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 28243: acqui folder
Bug-28243-acqui-folder.patch (text/plain), 19.96 KB, created by
Wainui Witika-Park
on 2021-06-10 00:23:13 UTC
(
hide
)
Description:
Bug 28243: acqui folder
Filename:
MIME Type:
Creator:
Wainui Witika-Park
Created:
2021-06-10 00:23:13 UTC
Size:
19.96 KB
patch
obsolete
>From b4a0fd9021165fccfbb7ad5045a58dc4e6f53f30 Mon Sep 17 00:00:00 2001 >From: Wainui Witika-Park <wainuiwitikapark@catalyst.net.nz> >Date: Wed, 9 Jun 2021 09:05:40 +0000 >Subject: [PATCH] Bug 28243: acqui folder > >Added captions to tables for accessibility purposes, some of these >include class="sr-only" so that they are not visible, but they are still present for those who use screen-readers. > >To test: >1) Apply patch and dependencies >2) Go to Staff Client >3) Go to Acquisitions >4) Go to each of the pages/files and check if they have tables >5) Check that each of the tables have captions to describe them (unless they have special heading tags instead) - some of these will be >visible on the page and some will be hidden with "sr-only" > >Sponsored-by: Catalyst IT >--- > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/acqui-home.tt | 1 + > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/addorderiso2709.tt | 1 + > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt | 4 ++-- > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basketgroup.tt | 2 ++ > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/booksellers.tt | 1 + > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/duplicate_orders.tt | 1 + > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/edifactmsgs.tt | 1 + > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/histsearch.tt | 3 ++- > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoice-files.tt | 1 + > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoice.tt | 4 +++- > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoices.tt | 1 + > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/lateorders.tt | 1 + > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt | 2 +- > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/newordersubscription.tt | 1 + > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/newordersuggestion.tt | 1 + > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/ordered.tt | 1 + > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/orderreceive.tt | 4 ++-- > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt | 3 ++- > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcels.tt | 2 ++ > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/spent.tt | 1 + > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/supplier.tt | 2 +- > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/transferorder.tt | 3 +++ > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/uncertainprice.tt | 1 + > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/z3950_search.tt | 1 + > 24 files changed, 34 insertions(+), 9 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/acqui-home.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/acqui-home.tt >index 8b3f483b16..7edcba78f4 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/acqui-home.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/acqui-home.tt >@@ -104,6 +104,7 @@ > > <table id="accounts"> > >+ <caption class="sr-only">Accounts</caption> > <thead> > <tr> > <th>Active</th> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/addorderiso2709.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/addorderiso2709.tt >index ce25456335..ac502033be 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/addorderiso2709.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/addorderiso2709.tt >@@ -393,6 +393,7 @@ Batch list > <div> > <h1>Choose the file to add to the basket</h1> > <table id="files"> >+ <caption class="sr-only">Files</caption> > <thead> > <tr> > <th>File name</th> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt >index f2d33d696c..370ae03dd7 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt >@@ -414,8 +414,8 @@ > > <div id="acqui_basket_content"> > [% IF ( books_loop ) %] >- <h2>Orders</h2> > <table id="orders"> >+ <caption><h2>Orders</h2></caption> > <thead> > <tr> > <th>No.</th> >@@ -617,8 +617,8 @@ > </div> <!-- /#acqui_basket_content --> > [% IF (cancelledorders_loop) %] > <div id="cancelledorders"> >- <h2>Cancelled orders</h2> > <table id="cancelledorderst"> >+ <caption><h2>Cancelled orders</h2></caption> > <thead> > <tr> > <th>No.</th> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basketgroup.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basketgroup.tt >index 04574c1061..bc0b848e35 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basketgroup.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basketgroup.tt >@@ -333,6 +333,7 @@ fieldset.various li { > </ul> > <div id="opened"> > <table id="basket_group_opened"> >+ <caption class="sr-only">Opened</caption> > <thead> > <tr> > <th>Name</th> >@@ -372,6 +373,7 @@ fieldset.various li { > </div> > <div id="closed"> > <table id="basket_group_closed"> >+ <caption class="sr-only">Closed</caption> > <thead> > <tr> > <th>Name</th> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/booksellers.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/booksellers.tt >index 1c5f14ddcb..52e8de6c72 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/booksellers.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/booksellers.tt >@@ -96,6 +96,7 @@ > <div class="baskets"> > [% IF ( supplier.loop_basket.size ) %] > <table class="baskets"> >+ <caption class="sr-only">Baskets</caption> > <thead> > <tr> > <th>No.</th> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/duplicate_orders.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/duplicate_orders.tt >index f70b33a1ce..e36780f632 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/duplicate_orders.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/duplicate_orders.tt >@@ -265,6 +265,7 @@ Acquisitions › Koha > [% ELSIF op == 'duplication_done' %] > [% IF new_orders %] > <table id="table_neworders"> >+ <caption class="sr-only">New orders</caption> > <thead> > <tr> > <th>Order line</th> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/edifactmsgs.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/edifactmsgs.tt >index 3ea4eafb04..4cf6ee591d 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/edifactmsgs.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/edifactmsgs.tt >@@ -38,6 +38,7 @@ > > [% IF ( messages ) %] > <table id="edi_msgs"> >+ <caption class="sr-only">Messages</caption> > <thead> > <tr> > <th>Type</th> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/histsearch.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/histsearch.tt >index 645593c211..0a59819b8e 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/histsearch.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/histsearch.tt >@@ -59,9 +59,10 @@ > [% END %] > > >- [% IF ( order_loop ) %]<h2>Search results</h2> >+ [% IF ( order_loop ) %] > <div id="acqui_histsearch"> > <table id="histsearcht"> >+ <caption><h2>Search results</h2></caption> > <thead> > <tr> > <th>Order line (parent)</th> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoice-files.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoice-files.tt >index a9077ed480..adfd0fd1e2 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoice-files.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoice-files.tt >@@ -49,6 +49,7 @@ > [% END %] > [% IF files %] > <table id="invoice_files_details_table"> >+ <caption class="sr-only">Details of invoice files</caption> > <thead> > <tr> > <th>Name</th> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoice.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoice.tt >index 2160d44369..1a949665c7 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoice.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoice.tt >@@ -157,6 +157,7 @@ > <input type="hidden" name="invoiceid" value="[% invoiceid | html %]" /> > [% IF (adjustments && adjustments.count > 0) %] > <table id="invoice_adj_table"> >+ <caption class="sr-only">Adjustments</caption> > <tr> > <th>Id</th> > <th>Amount</th> >@@ -342,6 +343,7 @@ > Show all details > </label> > <table id="orderst"> >+ <caption class="sr-only">Orders</caption> > <thead> > <tr> > <th class="anti-the">Summary</th> >@@ -439,8 +441,8 @@ > [% END %] > [% IF ( (Koha.Preference('AcqEnableFiles')) && files ) %] > <br /> >- <h2>Files attached to invoice</h2> > <table id="invoice_files_table"> >+ <caption><h2>Files attached to invoice</h2></caption> > <thead> > <tr> > <th>Name</th> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoices.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoices.tt >index e6cb2afb38..ecad17e9cc 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoices.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoices.tt >@@ -49,6 +49,7 @@ > Show only subscriptions > </label> > <table id="[% tab | html %]resultst" class="result"> >+ <caption class="sr-only">Results</caption> > <thead> > <tr> > [% IF CAN_user_acquisition_merge_invoices %] >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/lateorders.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/lateorders.tt >index 8747e9f361..6e2dafc33e 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/lateorders.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/lateorders.tt >@@ -68,6 +68,7 @@ > <p><a id="CheckAll" href="#"><i class="fa fa-check"></i> Check all</a> <a id="CheckNone" href="#"><i class="fa fa-remove"></i> Uncheck all</a></p> > [% END %] > <table id="late_orders"> >+ <caption class="sr-only">Late orders</caption> > <thead> > <tr> > <th class="NoSort"></th> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt >index 6dd9526d36..d59da11a00 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt >@@ -282,9 +282,9 @@ > [% IF (AcqCreateItemOrdering) %] > > <div id="items_list" style="display:none"> >- <p><strong>Items list</strong></p> > <div style="width:100%;overflow:auto;"> > <table> >+ <caption>Items list</caption> > <thead> > <tr> > <th class="noExport">Actions</th> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/newordersubscription.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/newordersubscription.tt >index 2986089b95..ec6bcca47e 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/newordersubscription.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/newordersubscription.tt >@@ -49,6 +49,7 @@ > </label> > [% IF (subs_loop) %] > <table id="srlt"> >+ <caption class="sr-only">Subscriptions</caption> > <thead> > <tr> > <th>ISSN</th> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/newordersuggestion.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/newordersuggestion.tt >index 0946308afb..0f22363da3 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/newordersuggestion.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/newordersuggestion.tt >@@ -44,6 +44,7 @@ > [% IF ( suggestions_loop ) %] > <a href="#" id="show_only_mine">Show only mine</a> | <a href="#" id="show_all">Show all suggestions</a> > <table id="suggestionst"> >+ <caption class="sr-only">Suggestions</caption> > <thead> > <tr> > <th>Mine</th> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/ordered.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/ordered.tt >index f115cbd76c..fd77189c3a 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/ordered.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/ordered.tt >@@ -39,6 +39,7 @@ > <h1>Ordered - Fund: [% fund_code | html %]</h1> > > <table id="spent"> >+ <caption class="sr-only">Orders</caption> > <thead> > <tr> > <th class="anti-the"> Title </th> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/orderreceive.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/orderreceive.tt >index 0c2ea8ee67..dafb296dcd 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/orderreceive.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/orderreceive.tt >@@ -177,9 +177,9 @@ > </fieldset> > [% ELSIF (AcqCreateItem == 'receiving') %] > <div id="items_list" style="display:none"> >- <p><strong>Items list</strong></p> > <div style="width:100%;overflow:auto;"> > <table> >+ <caption>Items list</caption> > <thead> > <tr> > <th class="noExport">Actions</th> >@@ -219,9 +219,9 @@ > [% END %] > [% ELSIF (AcqCreateItem == 'ordering') %] > [% IF (order.items) %] >- <h2>Items</h2> > <div style="width:100%;overflow:auto"> > <table> >+ <caption><h2>Items</h2></caption> > <thead> > <tr> > <th>Receive?</th> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt >index 3bf342d5a3..a74b524ffe 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt >@@ -113,8 +113,8 @@ > </div> > [% UNLESS (invoiceclosedate) %] > <div id="acqui_receive_search"> >- <h2>Pending orders</h2> > <table id="pending_orders" class="table table-bordered table-striped"> >+ <caption><h2>Pending orders</h2></caption> > <thead> > <tr> > <th>Basket</th> >@@ -148,6 +148,7 @@ > [% IF ( loop_received ) %] > <form action="/cgi-bin/koha/acqui/parcel.pl" method="get" name="orderform"> > <table id="receivedt"> >+ <caption class="sr-only">Received</caption> > <thead> > <tr> > <th>Basket</th> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcels.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcels.tt >index a1621fe63f..32abcf5bb7 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcels.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcels.tt >@@ -50,6 +50,7 @@ > <div id="parcels_duplicate_invoice" class="dialog alert"> > <p>This invoice number has already been used. Would you like to receive on an existing invoice?</p> > <table> >+ <caption class="sr-only">Duplicate invoices</caption> > <thead><tr><th>Invoice no.</th><th>Shipment date</th><th></th></tr></thead> > <tbody> > [% FOREACH invoice IN duplicate_invoices %] >@@ -80,6 +81,7 @@ > <!-- Search Results Table --> > > <table class="small" id="parcelst"> >+ <caption class="sr-only">Parcels</caption> > <thead> > <tr> > <th>Line</th> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/spent.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/spent.tt >index 5eca17fe30..42ec6128e7 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/spent.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/spent.tt >@@ -39,6 +39,7 @@ > <h1>Spent - Fund: [% fund_code | html %]</h1> > > <table id="spent"> >+ <caption class="sr-only">Spent</caption> > <thead> > <tr> > <th class="anti-the">Title</th> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/supplier.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/supplier.tt >index d15842a45e..d9339d1fa3 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/supplier.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/supplier.tt >@@ -382,8 +382,8 @@ > > [% IF ( contracts ) %] > <div id="supplier-contracts"> >- <h2>Contract(s)</h2> > <table id="contractst"> >+ <caption><h2>Contract(s)</h2></caption> > <thead> > <tr> > <th scope="col">Name</th> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/transferorder.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/transferorder.tt >index e17a1fa3e9..f9543fc168 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/transferorder.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/transferorder.tt >@@ -39,6 +39,7 @@ > <h2>Baskets for [% booksellertoname | html %]</h2> > [% IF ( basketsloop ) %] > <table> >+ <caption class="sr-only">Baskets</caption> > <thead> > <tr> > <th>Basket (#)</th> >@@ -65,6 +66,7 @@ > [% IF ( ordersloop ) %] > <h2>Orders for [% booksellerfromname | html %]</h2> > <table> >+ <caption class="sr-only">Orders</caption> > <thead> > <tr> > <th>[% tp('noun', 'Order') | html %]</th> >@@ -101,6 +103,7 @@ > [% END %] > [% IF ( booksellersloop ) %] > <table> >+ <caption class="sr-only">Vendors</caption> > <thead> > <tr> > <th>Name</th> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/uncertainprice.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/uncertainprice.tt >index c6c40ceda5..86ee5dd5e5 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/uncertainprice.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/uncertainprice.tt >@@ -98,6 +98,7 @@ > <input type="hidden" name="booksellerid" value="[% booksellerid | html %]" /> > <input type="hidden" name="op" value="validate" /> > <table id="uncertainpricet"> >+ <caption class="sr-only">Uncertain price orders</caption> > <thead> > <tr> > <th>Basket</th> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/z3950_search.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/z3950_search.tt >index ed3aa75ca4..9f90354521 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/z3950_search.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/z3950_search.tt >@@ -144,6 +144,7 @@ tr.selected { background-color : #FFFFCC; } tr.selected td { background-color : > > [% IF ( breeding_loop ) %] > <table id="resultst"> >+ <caption class="sr-only">Results</caption> > <thead><tr> > <th>Server</th> > <th>Title</th> >-- >2.11.0
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 28243
: 121750 |
121895
|
121896
|
121898
|
121899
|
121900
|
121901
|
121902
|
121903
|
121904
|
121905
|
121942
|
121943
|
121944
|
121945
|
121946
|
121947
|
121948
|
121949
|
121950
|
121956
|
121957
|
121958
|
121959
|
121973
|
121974
|
122019
|
122020
|
122021