Bug 33147: Use template wrapper for breadcrumbs: Catalog part 2
This patch updates several catalog templates so that they use the new WRAPPER for displaying breadcrumbs. To test, apply the patch and test each page and its variations. Breadcrumbs should look correct, and each link should be correct. - Catalog search results - Search history (from the user menu in the header) - Bibliographic detail page -> Rota (must have StockRotation enabled). Signed-off-by: Andrew Auld <andrew.auld@ptfs-europe.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
parent
aa22e9fb48
commit
2cf9efa3b6
3 changed files with 34 additions and 48 deletions
|
@ -24,23 +24,23 @@
|
|||
[% SyndeticsCovers = Koha.Preference('SyndeticsEnabled') && Koha.Preference('SyndeticsCoverImages') %]
|
||||
|
||||
[% WRAPPER 'sub-header.inc' %]
|
||||
<nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumb">
|
||||
<ol>
|
||||
<li>
|
||||
<a href="/cgi-bin/koha/mainpage.pl">Home</a>
|
||||
</li>
|
||||
<li>
|
||||
[% WRAPPER breadcrumbs %]
|
||||
[% WRAPPER breadcrumb_item %]
|
||||
<a href="/cgi-bin/koha/catalogue/search.pl">Catalog</a>
|
||||
</li>
|
||||
<li>
|
||||
<a href="#" aria-current="page">
|
||||
Search [% IF ( query_desc ) %] for '[% query_desc | html %]'[% END %]
|
||||
[% IF ( limit_desc ) %] with limit(s): '[% limit_desc | html %]'[% END %]
|
||||
</a>
|
||||
</li>
|
||||
</ol>
|
||||
</nav>
|
||||
[% END %]
|
||||
[% END %]
|
||||
[% WRAPPER breadcrumb_item bc_active= 1 %]
|
||||
[% IF ( query_desc ) %]
|
||||
[% IF ( limit_desc ) %]
|
||||
<span>Search for '[% query_desc | html %]' with limit(s): '[% limit_desc | html %]'</span>
|
||||
[% ELSE %]
|
||||
<span>Search for '[% query_desc | html %]'</span>
|
||||
[% END %]
|
||||
[% ELSE %]
|
||||
<span>Search</span>
|
||||
[% END %]
|
||||
[% END %]
|
||||
[% END #/ WRAPPER breadcrumbs %]
|
||||
[% END #/ WRAPPER sub-header.inc %]
|
||||
|
||||
<div class="main container-fluid">
|
||||
<div class="row">
|
||||
|
|
|
@ -15,22 +15,15 @@
|
|||
[% END %]
|
||||
|
||||
[% WRAPPER 'sub-header.inc' %]
|
||||
<nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumb">
|
||||
<ol>
|
||||
<li>
|
||||
<a href="/cgi-bin/koha/mainpage.pl">Home</a>
|
||||
</li>
|
||||
<li>
|
||||
[% WRAPPER breadcrumbs %]
|
||||
[% WRAPPER breadcrumb_item %]
|
||||
<a href="/cgi-bin/koha/catalogue/search.pl">Catalog</a>
|
||||
</li>
|
||||
<li>
|
||||
<a href="#" aria-current="page">
|
||||
Search history
|
||||
</a>
|
||||
</li>
|
||||
</ol>
|
||||
</nav>
|
||||
[% END %]
|
||||
[% END %]
|
||||
[% WRAPPER breadcrumb_item bc_active= 1 %]
|
||||
<span>Search history</span>
|
||||
[% END %]
|
||||
[% END #/ WRAPPER breadcrumbs %]
|
||||
[% END #/ WRAPPER sub-header.inc %]
|
||||
|
||||
<div class="container-fluid">
|
||||
<div class="row">
|
||||
|
|
|
@ -14,25 +14,18 @@
|
|||
[% END %]
|
||||
|
||||
[% WRAPPER 'sub-header.inc' %]
|
||||
<nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumb">
|
||||
<ol>
|
||||
<li>
|
||||
<a href="/cgi-bin/koha/mainpage.pl">Home</a>
|
||||
</li>
|
||||
<li>
|
||||
[% WRAPPER breadcrumbs %]
|
||||
[% WRAPPER breadcrumb_item %]
|
||||
<a href="/cgi-bin/koha/catalogue/search.pl">Catalog</a>
|
||||
</li>
|
||||
<li>
|
||||
[% END %]
|
||||
[% WRAPPER breadcrumb_item %]
|
||||
[% INCLUDE 'biblio-title.inc' link = 1 %]
|
||||
</li>
|
||||
<li>
|
||||
<a href="#" aria-current="page">
|
||||
Stock rotation details
|
||||
</a>
|
||||
</li>
|
||||
</ol>
|
||||
</nav>
|
||||
[% END %]
|
||||
[% END %]
|
||||
[% WRAPPER breadcrumb_item bc_active= 1 %]
|
||||
<span>Stock rotation details</span>
|
||||
[% END %]
|
||||
[% END #/ WRAPPER breadcrumbs %]
|
||||
[% END #/ WRAPPER sub-header.inc %]
|
||||
|
||||
<div class="main container-fluid">
|
||||
<div class="row">
|
||||
|
|
Loading…
Reference in a new issue