gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 33599: Use template wrapper for breadcrumbs: Various 

This patch updates various 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.

- ILL requests
  - New ill request
  - Manage ill request
- Bibliographic record ->
  - Place hold
  - Place multiple holds
- Tools ->
  - Comments awaiting moderation
  - Approved comments
- Suggestions
  - New suggestion
  - View suggestion
  - Edit suggestion
- Lists
  - Public Lists
  - Private lists
  - New list
  - Edit list
  - Transfer list
  - View list contents

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit 50f387876e)
Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
This commit is contained in:
Owen Leonard 2023-04-24 11:25:21 +00:00 committed by Matt Blenkinsop
parent 2e55b07092
commit 386cdb6c57
5 changed files with 110 additions and 182 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

44
koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt
View File

@ -19,40 +19,28 @@
[% 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>
[% WRAPPER breadcrumbs %]
[% IF query_type == 'create' %]
<li>
[% WRAPPER breadcrumb_item %]
<a href="/cgi-bin/koha/ill/ill-requests.pl">ILL requests</a>
</li>
<li>
<a href="#" aria-current="page">
New request
</a>
</li>
[% END %]
[% WRAPPER breadcrumb_item bc_active= 1 %]
<span>New request</span>
[% END %]
[% ELSIF query_type == 'illview' %]
<li>
[% WRAPPER breadcrumb_item %]
<a href="/cgi-bin/koha/ill/ill-requests.pl">ILL requests</a>
</li>
<li>
<a href="#" aria-current="page">
Manage request [% request.id_prefix _ request.illrequest_id | html %]
</a>
</li>
[% END %]
[% WRAPPER breadcrumb_item bc_active= 1 %]
<span>Manage request [% request.id_prefix _ request.illrequest_id | html %]</span>
[% END %]
[% ELSE %]
<li>
<a href="#" aria-current="page">
ILL requests
</a>
</li>
[% WRAPPER breadcrumb_item bc_active= 1 %]
<span>ILL requests</span>
[% END %]
[% END %]
</ol>
</nav>
[% END %]
[% END #/ WRAPPER breadcrumbs %]
[% END #/ WRAPPER sub-header.inc %]
<div class="main container-fluid">
<div class="row">

71
koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
View File

@ -98,55 +98,36 @@
[% END %]
[% WRAPPER 'sub-header.inc' %]
[% UNLESS ( multi_hold ) %]
<nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumb">
<ol>
<li>
<a href="/cgi-bin/koha/mainpage.pl">Home</a>
</li>
<li>
<a href="/cgi-bin/koha/catalogue/search.pl">Catalog</a>
</li>
<li>
[% WRAPPER breadcrumbs %]
[% WRAPPER breadcrumb_item %]
<a href="/cgi-bin/koha/catalogue/search.pl">Catalog</a>
[% END %]
[% UNLESS ( multi_hold ) %]
[% WRAPPER breadcrumb_item %]
[% INCLUDE 'biblio-title.inc' link =1 %]
</li>
<li>
<a href="#" aria-current="page">
Place a hold
</a>
</li>
</ol>
</nav>
[% ELSE %]
<nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumb">
<ol>
<li>
<a href="/cgi-bin/koha/mainpage.pl">Home</a>
</li>
<li>
<a href="/cgi-bin/koha/catalogue/search.pl">Catalog</a>
</li>
<li>
[% IF ( patron ) %]
<a href="#" aria-current="page">
Place holds
</a>
[% END %]
[% WRAPPER breadcrumb_item bc_active= 1 %]
<span>Place a hold</span>
[% END %]
[% ELSE %]
[% IF ( patron ) %]
[% WRAPPER breadcrumb_item bc_active= 1 %]
<span>Place holds</span>
[% END %]
[% ELSE %]
[% IF clubcount %]
[% WRAPPER breadcrumb_item bc_active= 1 %]
<span>Search patrons or clubs</span>
[% END %]
[% ELSE %]
[% IF clubcount %]
<a href="#" aria-current="page">
Search patrons or clubs
</a>
[% ELSE %]
<a href="#" aria-current="page">
Search patrons
</a>
[% WRAPPER breadcrumb_item bc_active= 1 %]
<span>Search patrons</span>
[% END %]
[% END %]
</li>
</ol>
</nav>
[% END # UNLESS multi_hold %]
[% END %]
[% END %]
[% END # UNLESS multi_hold %]
[% END #/ WRAPPER breadcrumbs %]
[% END #/ WRAPPER sub-header.inc %]
<div class="main container-fluid">
<div class="row">

37
koha-tmpl/intranet-tmpl/prog/en/modules/reviews/reviewswaiting.tt
View File

@ -11,29 +11,24 @@
[% 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/tools/tools-home.pl">Tools</a>
</li>
<li>
[% END %]
[% WRAPPER breadcrumb_item %]
<a href="/cgi-bin/koha/reviews/reviewswaiting.pl">Comments</a>
</li>
<li>
<a href="#" aria-current="page">
[% IF ( status ) %]
<span>Approved comments</span>
[% ELSE %]
<span>Comments awaiting moderation</span>
[% END %]
</a>
</li>
</ol>
</nav>
[% END %]
[% END %]
[% IF ( status ) %]
[% WRAPPER breadcrumb_item bc_active= 1 %]
<span>Approved comments</span>
[% END %]
[% ELSE %]
[% WRAPPER breadcrumb_item bc_active= 1 %]
<span>Comments awaiting moderation</span>
[% END %]
[% END %]
[% END #/ WRAPPER breadcrumbs %]
[% END #/ WRAPPER sub-header.inc %]
<div class="main container-fluid">
<div class="row">

62
koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt
View File

@ -59,54 +59,36 @@
[% 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/acqui/acqui-home.pl">Acquisitions</a>
</li>
[% END %]
[% IF ( op == 'save' || op == 'show' ) %]
[% WRAPPER breadcrumb_item %]
<a href="/cgi-bin/koha/suggestion/suggestion.pl">Suggestions</a>
[% END %]
[% END %]
[% IF op == 'save' %]
[% IF ( suggestionid ) %]
<li>
<a href="/cgi-bin/koha/suggestion/suggestion.pl">Suggestions</a>
</li>
<li>
<a href="#" aria-current="page">
Edit suggestion #[% suggestionid | html %]
</a>
</li>
[% WRAPPER breadcrumb_item bc_active= 1 %]
<span>Edit suggestion #[% suggestionid | html %]</span>
[% END %]
[% ELSE %]
<li>
<a href="/cgi-bin/koha/suggestion/suggestion.pl">Suggestions</a>
</li>
<li>
<a href="#" aria-current="page">
Add suggestion
</a>
</li>
[% WRAPPER breadcrumb_item bc_active= 1 %]
<span>Add suggestion</span>
[% END %]
[% END %]
[% ELSIF ( op == 'show' ) %]
<li>
<a href="/cgi-bin/koha/suggestion/suggestion.pl">Suggestions</a>
</li>
<li>
<a href="#" aria-current="page">
Show suggestion #[% suggestionid | html %]
</a>
</li>
[% WRAPPER breadcrumb_item bc_active= 1 %]
<span>Show suggestion #[% suggestionid | html %]</span>
[% END %]
[% ELSE %]
<li>
<a href="#" aria-current="page">
Suggestions management
</a>
</li>
[% WRAPPER breadcrumb_item bc_active= 1 %]
<span>Suggestions management</span>
[% END %]
[% END %]
</ol>
</nav> <!-- /.breadcrumbs -->
[% END %]
[% END #/ WRAPPER breadcrumbs %]
[% END #/ WRAPPER sub-header.inc %]
[% IF ( op == 'show' ) %]
<div class="main container-fluid">

78
koha-tmpl/intranet-tmpl/prog/en/modules/virtualshelves/shelves.tt
View File

@ -61,83 +61,65 @@
[% 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>
[% WRAPPER breadcrumbs %]
[% IF op != 'list' %]
<li>
[% WRAPPER breadcrumb_item %]
<a href="/cgi-bin/koha/virtualshelves/shelves.pl">Lists</a>
</li>
[% END %]
[% ELSE %]
<li>
<a href="#" aria-current="page">
Lists
</a>
</li>
[% WRAPPER breadcrumb_item bc_active= 1 %]
<span>Lists</span>
[% END %]
[% END %]
[% IF shelf AND shelf.is_private %]
[% IF op == 'view' OR op == 'edit_form' %]
<li>
[% WRAPPER breadcrumb_item %]
<a href="/cgi-bin/koha/virtualshelves/shelves.pl?op=list&amp;public=0">Your lists</a>
</li>
[% END %]
[% ELSE %]
<li>
<a href="#" aria-current="page">
Your lists
</a>
</li>
[% WRAPPER breadcrumb_item bc_active= 1 %]
<span>Your lists</span>
[% END %]
[% END %]
[% ELSIF shelf AND shelf.is_public %]
[% IF op == 'view' %]
<li>
[% WRAPPER breadcrumb_item %]
<a href="/cgi-bin/koha/virtualshelves/shelves.pl?op=list&amp;public=1">Public lists</a>
</li>
[% END %]
[% ELSE %]
<li>
<a href="#" aria-current="page">
Public lists
</a>
</li>
[% WRAPPER breadcrumb_item bc_active= 1 %]
<span>Public lists</span>
[% END %]
[% END %]
[% END %]
[% IF op == 'view' %]
<li>
<a href="#" aria-current="page">
Contents of <em>[% shelf.shelfname | html %]</em>
</a>
</li>
[% WRAPPER breadcrumb_item bc_active= 1 %]
<span>Contents of <em>[% shelf.shelfname | html %]</em></span>
[% END %]
[% END %]
[% IF op == 'add_form' %]
<li>
<a href="#" aria-current="page">
Create new list
</a>
</li>
[% WRAPPER breadcrumb_item bc_active= 1 %]
<span>Create new list</span>
[% END %]
[% END %]
[% IF op == 'edit_form' %]
<li>
<a href="#" aria-current="page">
Edit list <em>[% shelf.shelfname | html %]</em>
</a>
</li>
[% WRAPPER breadcrumb_item bc_active= 1 %]
<span>Edit list <em>[% shelf.shelfname | html %]</em></span>
[% END %]
[% END %]
[% IF op == 'transfer' %]
<li>
<a href="#" aria-current="page">Transfer list <em>[% shelf.shelfname | html %]</em></a>
</li>
[% WRAPPER breadcrumb_item bc_active= 1 %]
<span>Transfer list <em>[% shelf.shelfname | html %]</em></span>
[% END %]
[% END %]
</ol>
</nav>
[% END %]
[% END #/ WRAPPER breadcrumbs %]
[% END #/ WRAPPER sub-header.inc %]
<div class="main container-fluid">
<div class="row">