Bugzilla – Attachment 13030 Details for
Bug 8712
We should use WAI-ARIA guidelines to help with accessibility -OMNIBUS bug
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 8712
Bug-8712.patch (text/plain), 8.46 KB, created by
Srikanth Dhondi
on 2012-10-25 04:21:37 UTC
(
hide
)
Description:
Bug 8712
Filename:
MIME Type:
Creator:
Srikanth Dhondi
Created:
2012-10-25 04:21:37 UTC
Size:
8.46 KB
patch
obsolete
>From ee186dd3d12a80b79a2c96ef40095270039fa2b5 Mon Sep 17 00:00:00 2001 >From: Srikanth Dhondi <srikanth@catalyst.net.nz> >Date: Thu, 25 Oct 2012 17:20:13 +1300 >Subject: [PATCH] Bug 8712 > >Made changes to advances search page to make it more accessibility friendly >--- > koha-tmpl/opac-tmpl/prog/en/includes/masthead.inc | 7 +- > .../opac-tmpl/prog/en/modules/opac-advsearch.tt | 89 ++++++++++++++------ > 2 files changed, 69 insertions(+), 27 deletions(-) > >diff --git a/koha-tmpl/opac-tmpl/prog/en/includes/masthead.inc b/koha-tmpl/opac-tmpl/prog/en/includes/masthead.inc >index 005a230..1d7bf35 100644 >--- a/koha-tmpl/opac-tmpl/prog/en/includes/masthead.inc >+++ b/koha-tmpl/opac-tmpl/prog/en/includes/masthead.inc >@@ -145,7 +145,7 @@ > No public lists > [% END %] > [% IF ( opacuserlogin ) %] >- <h4>Your lists</h4> >+ <h4>Your lists</h4> > [% IF ( loggedinusername ) %] > [% IF ( barshelves ) %] > <ul class="first-of-type"> >@@ -157,7 +157,7 @@ > [% ELSE %] > <ul class="first-of-type"> > <li>No private lists</li> >- <li class="yuimenuitem"><a class="yuimenuitemlabel" href="/cgi-bin/koha/opac-shelves.pl?display=privateshelves">[New list]</a></li></ul> >+ <li class="yuimenuitem"><a class="yuimenuitemlabel" href="/cgi-bin/koha/opac-shelves.pl?display=privateshelves">[Create new list]</a></li></ul> > [% END %] > [% ELSE %] > <ul class="first-of-type"><li><a href="/cgi-bin/koha/opac-user.pl">Log in to create your own lists</a></li></ul> >@@ -165,6 +165,7 @@ > [% END %] > </div> > </div><!-- /listmenu /virtualshelves --> >+ > [% END %] > <div id="listsDetails"></div> > </div> >@@ -173,7 +174,7 @@ > [% IF ( searchdesc ) %]<p>[% IF ( total ) %]<h3 class="heading-inline">Your search returned [% total |html %] results.</h3> [% IF ( related ) %] (related searches: [% FOREACH relate IN related %][% relate.related_search %][% END %]). [% END %] > <a href="[% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?[% query_cgi |html %][% limit_cgi |html %]&count=[% countrss |html %]&sort_by=acqdate_dsc&format=rss2" class="rsssearchlink"><img src="/opac-tmpl/prog/images/feed-icon-16x16.png" alt="Subscribe to this search" title="Subscribe to this search" border="0" class="rsssearchicon"/></a> > [% ELSE %] >-<strong>No results found!</strong> >+<h3>No results found!</h3> > <p> > [% IF ( searchdesc ) %] > No results found for that in [% LibraryName %] catalog. <a href="[% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?[% query_cgi |html %][% limit_cgi |html %]&format=rss2" class="rsssearchlink"><img src="/opac-tmpl/prog/images/feed-icon-16x16.png" alt="Subscribe to this search" title="Subscribe to this search" border="0" class="rsssearchicon"/></a> >diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-advsearch.tt b/koha-tmpl/opac-tmpl/prog/en/modules/opac-advsearch.tt >index 515d954..0bbfaf4 100644 >--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-advsearch.tt >+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-advsearch.tt >@@ -55,17 +55,54 @@ > [% FOREACH search_box IN search_boxes_loop %] > <p> > [% IF ( expanded_options ) %] >+ [% IF loop.index == 0 %] >+ <span> </span> >+ [% END %] >+ > [% IF ( search_box.boolean ) %] >- <select name="op"> >- <option value="and" selected="selected">and</option> >- <option value="or">or</option> >- <option value="not">not</option> >- </select> >+ <select name="op"> >+ <option value="and" selected="selected">and</option> >+ <option value="or">or</option> >+ <option value="not">not</option> >+ </select> > [% END %] >+ [% ELSE %] >+ <label for="foand_[% loop.index %]"> >+ [% IF loop.index == 0 %] >+ >+ [% ELSE %] >+ and >+ [% END %] >+ </label> > [% END %] >- [% IF ( expanded_options ) %][% left_content %][% END %] >- <select name="idx"> >- <option value="kw">Keyword</option> >+ [% IF ( expanded_options ) %][% left_content %][% END %] >+ <select id="foand_[% loop.index %]" name="idx"> >+ [% IF loop.index == 0 %] >+ <option value="kw">Keyword</option> >+ [% ELSIF loop.index == 1 %] >+ <option value="ti">Title</option> >+ [% IF ( expanded_options ) %] >+ <option value="ti,phr"> Title phrase</option> >+ <option value="se,wrdl">Series title</option> >+ <option value="callnum">Call number</option> >+ <option value="location">Shelving location</option> >+ [% END %] >+ [% ELSIF loop.index == 2 %] >+ <option value="au,wrdl">Author</option> >+ [% IF ( expanded_options ) %] >+ <option value="au,phr"> Author phrase</option> >+ <option value="cpn,wrdl"> Corporate name</option> >+ <option value="cfn,wrdl"> Conference name</option> >+ <option value="cfn,phr"> Conference name phrase</option> >+ <option value="pn,wrdl"> Personal name</option> >+ <option value="pn,phr"> Personal name phrase</option> >+ [% END %] >+ [% END %] >+ >+ [% IF loop.index != 0 %] >+ <option value="kw">Keyword</option> >+ [% END %] >+ > <option value="su,wrdl">Subject</option> > [% IF ( expanded_options ) %] > <option value="su,phr"> Subject phrase</option> >@@ -73,23 +110,27 @@ > <option value="su-na"> Subject and narrower terms</option> > <option value="su-rl"> Subject and related terms</option> > [% END %] >- <option value="ti">Title</option> >-[% IF ( expanded_options ) %] >- <option value="ti,phr"> Title phrase</option> >- <option value="se,wrdl">Series title</option> >- <option value="callnum">Call number</option> >- <option value="location">Shelving location</option> >-[% END %] >- <option value="au,wrdl">Author</option> >+ [% IF loop.index != 1 %] >+ <option value="ti">Title</option> >+ [% IF ( expanded_options ) %] >+ <option value="ti,phr"> Title phrase</option> >+ <option value="se,wrdl">Series title</option> >+ <option value="callnum">Call number</option> >+ <option value="location">Shelving location</option> >+ [% END %] >+ [% END %] > >-[% IF ( expanded_options ) %] >- <option value="au,phr"> Author phrase</option> >- <option value="cpn,wrdl"> Corporate name</option> >- <option value="cfn,wrdl"> Conference name</option> >- <option value="cfn,phr"> Conference name phrase</option> >- <option value="pn,wrdl"> Personal name</option> >- <option value="pn,phr"> Personal name phrase</option> >-[% END %] >+ [% IF loop.index != 2 %] >+ <option value="au,wrdl">Author</option> >+ [% IF ( expanded_options ) %] >+ <option value="au,phr"> Author phrase</option> >+ <option value="cpn,wrdl"> Corporate name</option> >+ <option value="cfn,wrdl"> Conference name</option> >+ <option value="cfn,phr"> Conference name phrase</option> >+ <option value="pn,wrdl"> Personal name</option> >+ <option value="pn,phr"> Personal name phrase</option> >+ [% END %] >+ [% END %] > [% IF ( expanded_options ) %] > <option value="nt">Notes/Comments</option> > <option value="curriculum">Curriculum</option> >-- >1.7.9.5
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 8712
:
12869
|
12871
|
12891
|
13030
|
13065
|
13088
|
13090
|
13091
|
13092
|
13093
|
13106
|
13107
|
14300
|
18784
|
18785