Attachment 158265 Details for Bug 34866 – Bug 34866: Use template wrapper for breadcrumbs: OPAC part 4

[patch] Bug 34866: Use template wrapper for breadcrumbs: OPAC part 4

Bug-34866-Use-template-wrapper-for-breadcrumbs-OPA.patch (text/plain), 25.69 KB, created by David Nind on 2023-11-02 14:49:25 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: David Nind

Created: 2023-11-02 14:49:25 UTC

Size: 25.69 KB

patch

obsolete

>From 7ce785bf03dd0f5e9ca2909d492e6d007f496239 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Thu, 21 Sep 2023 15:17:34 +0000
>Subject: [PATCH] Bug 34866: Use template wrapper for breadcrumbs: OPAC part 4
>
>This patch updates several OPAC templates so that they
>use a new WRAPPER for displaying breadcrumbs.
>
>Apply the patch and log in to the OPAC. Test each of the following pages
>and their variations. Breadcrumbs should look correct, and each link
>should be correct:
>
>- Recent comments
>- Most popular
>- Enable "PatronSelfRegistration" and set "PatronSelfRegistrationVerifyByEmail"
>  to "Don't require."
>  - In the OPAC, go to "Register here." Fill out the form and submit
>    your registration.
>  - Check the "Registration complete" page.
>- Set "PatronSelfRegistrationVerifyByEmail" to "Require"
>  - Submit another registration and check the "Confirm your
>    registration" page.
>- Visit the registration page passing an invalid token:
>  /cgi-bin/koha/opac-registration-verify.pl?token=foo
>  Check the "Registration invalid" page.
>- Enable the "ArticleRequests" system preference and log in to the
>  OPAC.
>  - Perform a search which will return results.
>    - Check the search results page
>    - View the detail page for one of the results.
>     - Click "Request article."
>    - Return to the detail page and click "Place hold."
>      - Complete each step of the hold process.
>- Enable the "OpacAllowSharingPrivateLists" system preference.
>  - Create a private list if necessary.
>  - Click the "Share" link
>- Locate a serial record and view the detail page.
>  - Click the "More details" link under the "Subscriptions" plan
>- Enable the system preference "suggestion."
>  - From the logged-in user summary page click "Purchase suggestions"
>    - Click the "New purchase suggestion" link
>- Enable the "AnonSuggestions" and "OPACViewOthersSuggestions" system
>  preferences and log out of the OPAC.
>  - View the purchase suggestions page.
>- Enable the "OpacCloud" system preference and go to the "Subject cloud"
>  page. (the feature doesn't need to be configured to test).
>- Populate the "RestrictedPageTitle" and "RestrictedPageContent" system
>  preferences and navigate directly to /cgi-bin/koha/opac-restrictedpage.pl
>  - Remove the contents of "RestrictedPageTitle" and check the page
>    again.
>- This patch also updates svc/suggestion.tt but I don't know how to test
>  it!
>
>Signed-off-by: David Nind <david@davidnind.com>
>---
> .../modules/opac-registration-confirmation.tt | 15 ++----
> .../modules/opac-registration-email-sent.tt   | 16 +++----
> .../en/modules/opac-registration-invalid.tt   | 19 ++++----
> .../en/modules/opac-request-article.tt        | 22 ++++-----
> .../bootstrap/en/modules/opac-reserve.tt      | 16 +++----
> .../en/modules/opac-restrictedpage.tt         | 17 +++----
> .../bootstrap/en/modules/opac-results.tt      | 47 +++++++++----------
> .../en/modules/opac-serial-issues.tt          | 22 ++++-----
> .../bootstrap/en/modules/opac-shareshelf.tt   | 26 ++++------
> .../bootstrap/en/modules/opac-showreviews.tt  | 16 +++----
> .../bootstrap/en/modules/opac-suggestions.tt  | 32 +++++++------
> .../bootstrap/en/modules/opac-tags_subject.tt | 16 +++----
> .../bootstrap/en/modules/opac-topissues.tt    | 16 +++----
> .../bootstrap/en/modules/svc/suggestion.tt    | 16 +++----
> 14 files changed, 126 insertions(+), 170 deletions(-)
>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-registration-confirmation.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-registration-confirmation.tt
>index dc5e3a7388..6d904f4f2e 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-registration-confirmation.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-registration-confirmation.tt
>@@ -13,16 +13,11 @@
> [% INCLUDE 'masthead.inc' %]
> 
> <div class="main">
>-    <nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumbs">
>-        <ol class="breadcrumb">
>-            <li class="breadcrumb-item">
>-                <a href="/cgi-bin/koha/opac-main.pl">Home</a>
>-            </li>
>-            <li class="breadcrumb-item active">
>-                <a href="#" aria-current="page">Registration complete</a>
>-            </li>
>-        </ol>
>-    </nav> 
>+    [% WRAPPER breadcrumbs %]
>+        [% WRAPPER breadcrumb_item bc_active= 1 %]
>+            <span>Registration complete</span>
>+        [% END %]
>+    [% END #/ WRAPPER breadcrumbs %]
> 
>     <div class="container-fluid">
>         <div class="row">
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-registration-email-sent.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-registration-email-sent.tt
>index 295da996aa..abbd92032a 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-registration-email-sent.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-registration-email-sent.tt
>@@ -1,3 +1,4 @@
>+[% USE raw %]
> [% USE Koha %]
> [% USE AdditionalContents %]
> [% SET OpacNav = AdditionalContents.get( location => "OpacNav", lang => lang, library => logged_in_user.branchcode || default_branch, blocktitle => 0 ) %]
>@@ -11,16 +12,11 @@
> [% INCLUDE 'masthead.inc' %]
> 
>     <div class="main">
>-        <nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumbs">
>-            <ol class="breadcrumb">
>-                <li class="breadcrumb-item">
>-                    <a href="/cgi-bin/koha/opac-main.pl">Home</a>
>-                </li>
>-                <li class="breadcrumb-item active">
>-                    <a href="#">Register a new account</a>
>-                </li>
>-            </ol>
>-        </nav> 
>+        [% WRAPPER breadcrumbs %]
>+            [% WRAPPER breadcrumb_item bc_active= 1 %]
>+                <span>Confirm your registration</span>
>+            [% END %]
>+        [% END #/ WRAPPER breadcrumbs %]
> 
>         <div class="container-fluid">
>             <div class="row">
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-registration-invalid.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-registration-invalid.tt
>index feede44fcf..3ee11b637c 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-registration-invalid.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-registration-invalid.tt
>@@ -1,3 +1,4 @@
>+[% USE raw %]
> [% USE Koha %]
> [% USE AdditionalContents %]
> [% SET OpacNav = AdditionalContents.get( location => "OpacNav", lang => lang, library => logged_in_user.branchcode || default_branch, blocktitle => 0 ) %]
>@@ -11,16 +12,14 @@
> [% INCLUDE 'masthead.inc' %]
> 
>     <div class="main">
>-        <nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumbs">
>-            <ol class="breadcrumb">
>-                <li class="breadcrumb-item">
>-                    <a href="/cgi-bin/koha/opac-main.pl">Home</a>
>-                </li>
>-                <li class="breadcrumb-item active">
>-                    <a href="#" aria-current="page">Register a new account</a>
>-                </li>
>-            </ol>
>-        </nav> 
>+        [% WRAPPER breadcrumbs %]
>+            [% WRAPPER breadcrumb_item %]
>+                <a href="/cgi-bin/koha/opac-memberentry.pl">Register a new account</a>
>+            [% END %]
>+            [% WRAPPER breadcrumb_item bc_active= 1 %]
>+                <span>Registration invalid</span>
>+            [% END %]
>+        [% END #/ WRAPPER breadcrumbs %]
> 
>         <div class="container-fluid">
>             <div class="row">
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-request-article.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-request-article.tt
>index 4ba935bb8d..1b6ef84443 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-request-article.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-request-article.tt
>@@ -1,3 +1,4 @@
>+[% USE raw %]
> [% USE Koha %]
> [% USE Branches %]
> [% USE ItemTypes %]
>@@ -17,19 +18,14 @@
> [% SET disclaimer = AdditionalContents.get( location => "ArticleRequestsDisclaimerText", lang => lang, library => logged_in_user.branchcode || default_branch ) %]
> 
> <div class="main">
>-    <nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumbs">
>-        <ol class="breadcrumb">
>-            <li class="breadcrumb-item">
>-                <a href="/cgi-bin/koha/opac-main.pl">Home</a>
>-            </li>
>-            <li class="breadcrumb-item">
>-                <a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% biblio.biblionumber | uri %]">Details for: [% biblio.title | html %]</a>
>-            </li>
>-            <li class="breadcrumb-item active">
>-                <a href="#" aria-current="page">Request article</a>
>-            </li>
>-        </ol>
>-    </nav> 
>+    [% WRAPPER breadcrumbs %]
>+        [% WRAPPER breadcrumb_item %]
>+            <a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% biblio.biblionumber | uri %]">Details for: [% biblio.title | html %]</a>
>+        [% END %]
>+        [% WRAPPER breadcrumb_item bc_active= 1 %]
>+            <span>Request article</span>
>+        [% END %]
>+    [% END #/ WRAPPER breadcrumbs %]
> 
>     <div class="container-fluid maincontent">
>         <div class="row">
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-reserve.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-reserve.tt
>index e2918332a3..7b1e9d3708 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-reserve.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-reserve.tt
>@@ -1,3 +1,4 @@
>+[% USE raw %]
> [% USE Koha %]
> [% USE Branches %]
> [% USE KohaDates %]
>@@ -14,16 +15,11 @@
> [% INCLUDE 'masthead.inc' %]
> 
> <div class="main">
>-    <nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumbs">
>-        <ol class="breadcrumb">
>-            <li class="breadcrumb-item">
>-                <a href="/cgi-bin/koha/opac-main.pl">Home</a>
>-            </li>
>-            <li class="breadcrumb-item active">
>-                <a href="#" aria-current="page">Placing a hold</a>
>-            </li>
>-        </ol>
>-    </nav> 
>+    [% WRAPPER breadcrumbs %]
>+        [% WRAPPER breadcrumb_item bc_active= 1 %]
>+            <span>Placing a hold</span>
>+        [% END %]
>+    [% END #/ WRAPPER breadcrumbs %]
> 
>     <div class="container">
>         <div id="holds" class="maincontent">
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-restrictedpage.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-restrictedpage.tt
>index b9fc832961..ebb0a37c5f 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-restrictedpage.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-restrictedpage.tt
>@@ -22,18 +22,15 @@
> [% INCLUDE 'masthead.inc' %]
> 
> <div class="main">
>-    <nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumbs">
>-        <ol class="breadcrumb">
>-            <li class="breadcrumb-item">
>-               <a href="/cgi-bin/koha/opac-main.pl">Home</a>
>-            </li>
>+    [% WRAPPER breadcrumbs %]
>+        [% WRAPPER breadcrumb_item bc_active= 1 %]
>             [% IF Koha.Preference('RestrictedPageTitle') %]
>-                <li class="breadcrumb-item active">
>-                    <a href="#" aria-current="page">[% Koha.Preference('RestrictedPageTitle') | html %]</a>
>-                </li>
>+                [% Koha.Preference('RestrictedPageTitle') | html %]
>+            [% ELSE %]
>+                <span>Restricted page</span>
>             [% END %]
>-        </ol>
>-    </nav> 
>+        [% END %]
>+    [% END #/ WRAPPER breadcrumbs %]
> 
>     <div class="container-fluid">
>         <div class="row">
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt
>index 323fb1e889..702a6a60fc 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt
>@@ -34,31 +34,30 @@
> [% INCLUDE 'masthead.inc' %]
> 
>     <div class="main">
>-        <nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumbs">
>-            <ol class="breadcrumb">
>-                <li class="breadcrumb-item">
>-                    <a href="/cgi-bin/koha/opac-main.pl">Home</a>
>-                </li>
>-                [% IF ( searchdesc ) %]
>-                    [% IF ( ReturnPath ) %]
>-                        <li class="breadcrumb-item">
>-                            <a href="[% ReturnPath | url %]">Advanced search</a>
>-                        </li>
>+        [% WRAPPER breadcrumbs %]
>+            [% IF ( searchdesc ) %]
>+                [% IF ( ReturnPath ) %]
>+                    [% WRAPPER breadcrumb_item %]
>+                        <a href="[% ReturnPath | url %]">Advanced search</a>
>                     [% END %]
>-                    <li class="breadcrumb-item active">
>-                        [% IF ( ms_value ) %]
>-                            <a href="#" aria-current="page" title="You searched [% IF ( query_desc ) %]for '[% query_desc | html %]'[% END %][% IF ( limit_desc ) %] with limit(s): '[% limit_desc | html %]'[% END %]">Results of search for '[% ms_value | html %]'[% IF ( pages > 1) %], page [% current_page_number | html %] of [% pages | html %][% END %]</a>
>-                        [% ELSE %]
>-                            <a href="#" aria-current="page" title="You searched [% IF ( query_desc ) %]for '[% query_desc | html %]'[% END %][% IF ( limit_desc ) %] with limit(s): '[% limit_desc | html %]'[% END %]">Search results</a>
>-                        [% END %]
>-                    </li>
>                 [% ELSE %]
>-                    <li class="breadcrumb-item active">
>-                        <a href="#" aria-current="page">You did not specify any search criteria</a>
>-                    </li>
>+                    [% WRAPPER breadcrumb_item %]
>+                        <a href="/cgi-bin/koha/opac-search.pl">Advanced search</a>
>+                    [% END %]
>                 [% END %]
>-            </ol>
>-        </nav> 
>+                [% WRAPPER breadcrumb_item bc_active= 1 %]
>+                    [% IF ( ms_value ) %]
>+                        <span title="You searched [% IF ( query_desc ) %]for '[% query_desc | html %]'[% END %][% IF ( limit_desc ) %] with limit(s): '[% limit_desc | html %]'[% END %]">Results of search for '[% ms_value | html %]'[% IF ( pages > 1) %], page [% current_page_number | html %] of [% pages | html %][% END %]</span>
>+                    [% ELSE %]
>+                        <span title="You searched [% IF ( query_desc ) %]for '[% query_desc | html %]'[% END %][% IF ( limit_desc ) %] with limit(s): '[% limit_desc | html %]'[% END %]">Search results</span>
>+                    [% END %]
>+                [% END %]
>+            [% ELSE %]
>+                [% WRAPPER breadcrumb_item bc_active= 1 %]
>+                    <span>You did not specify any search criteria</span>
>+                [% END %]
>+            [% END %]
>+        [% END #/ WRAPPER breadcrumbs %]
> 
>         [% UNLESS ( total ) %]
>             <div class="container-fluid">
>@@ -123,9 +122,9 @@
>                 [% END # / IF searchdesc %]
> 
>                 <div id="userresults">
>-                    [% IF ( DidYouMean ) %]
>+                    [% #IF ( DidYouMean ) %]
>                         <div id="didyoumean">Not what you expected? Check for <a href="/cgi-bin/koha/svc/suggestion?render=standalone&amp;q=[% querystring | uri %]">suggestions</a></div>
>-                    [% END %]
>+                    [% #END %]
> 
>                     [% IF ( koha_spsuggest ) %]
>                         Did you mean:
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-serial-issues.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-serial-issues.tt
>index 86dd33578c..834214a08f 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-serial-issues.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-serial-issues.tt
>@@ -1,3 +1,4 @@
>+[% USE raw %]
> [% USE Koha %]
> [% USE AdditionalContents %]
> [% USE AuthorisedValues %]
>@@ -19,19 +20,14 @@
> [% INCLUDE 'masthead.inc' %]
> 
>     <div class="main">
>-        <nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumbs">
>-            <ol class="breadcrumb">
>-                <li class="breadcrumb-item">
>-                    <a href="/cgi-bin/koha/opac-main.pl">Home</a>
>-                </li>
>-                <li class="breadcrumb-item">
>-                    <a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% biblionumber | uri %]">Details for [% bibliotitle | html %]</a>
>-                </li>
>-                <li class="breadcrumb-item active">
>-                    <a href="#" aria-current="page">Issues for a subscription</a>
>-                </li>
>-            </ol>
>-        </nav> 
>+        [% WRAPPER breadcrumbs %]
>+            [% WRAPPER breadcrumb_item %]
>+                <a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% biblionumber | uri %]">Details for [% bibliotitle | html %]</a>
>+            [% END %]
>+            [% WRAPPER breadcrumb_item bc_active= 1 %]
>+                <span>Issues for a subscription</span>
>+            [% END %]
>+        [% END #/ WRAPPER breadcrumbs %]
> 
>         <div class="container-fluid">
>             <div class="row">
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-shareshelf.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-shareshelf.tt
>index 5ea99a8915..98e1c39b91 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-shareshelf.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-shareshelf.tt
>@@ -1,3 +1,4 @@
>+[% USE raw %]
> [% USE Koha %]
> [% USE AdditionalContents %]
> [% SET OpacNav = AdditionalContents.get( location => "OpacNav", lang => lang, library => logged_in_user.branchcode || default_branch, blocktitle => 0 ) %]
>@@ -11,24 +12,17 @@
> [% INCLUDE 'bodytag.inc' bodyid='opac-shareshelf' bodyclass='scrollto' %]
> [% INCLUDE 'masthead.inc' %]
> <div class="main">
>-    <nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumbs">
>-        <ol class="breadcrumb">
>-            <li class="breadcrumb-item">
>-                <a href="/cgi-bin/koha/opac-main.pl">Home</a>
>-            </li>
>-            [% IF ( loggedinusername ) %]
>-                <li class="breadcrumb-item">
>-                    <a href="/cgi-bin/koha/opac-user.pl">[% INCLUDE 'patron-title.inc' patron = logged_in_user %]</a>
>-                </li>
>+        [% WRAPPER breadcrumbs %]
>+            [% WRAPPER breadcrumb_item %]
>+                <a href="/cgi-bin/koha/opac-user.pl">[% INCLUDE 'patron-title.inc' patron = logged_in_user %]</a>
>             [% END %]
>-            <li class="breadcrumb-item">
>+            [% WRAPPER breadcrumb_item %]
>                 <a href="/cgi-bin/koha/opac-shelves.pl">Lists</a>
>-            </li>
>-            <li class="breadcrumb-item active">
>-                <a href="#" aria-current="page">Share a list</a>
>-            </li>
>-        </ol> 
>-    </nav> 
>+            [% END %]
>+            [% WRAPPER breadcrumb_item bc_active= 1 %]
>+                <span>Share a list</span>
>+            [% END %]
>+        [% END #/ WRAPPER breadcrumbs %]
> 
>     <div class="container-fluid">
>         <div class="row">
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-showreviews.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-showreviews.tt
>index e307d688ef..517bcd82b0 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-showreviews.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-showreviews.tt
>@@ -1,3 +1,4 @@
>+[% USE raw %]
> [% USE Koha %]
> [% USE KohaDates %]
> [% USE AdditionalContents %]
>@@ -13,16 +14,11 @@
> [% INCLUDE 'masthead.inc' %]
> 
>     <div class="main">
>-        <nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumbs">
>-            <ol class="breadcrumb">
>-                <li class="breadcrumb-item">
>-                    <a href="/cgi-bin/koha/opac-main.pl">Home</a>
>-                </li>
>-                <li class="breadcrumb-item active">
>-                    <a href="#" aria-current="page">Recent comments</a>
>-                </li>
>-            </ol>
>-        </nav> 
>+        [% WRAPPER breadcrumbs %]
>+            [% WRAPPER breadcrumb_item bc_active= 1 %]
>+                <span>Recent comments</span>
>+            [% END %]
>+        [% END #/ WRAPPER breadcrumbs %]
> 
>         <div class="container-fluid">
>             <div class="row">
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt
>index 54d8998fb3..2997f20583 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt
>@@ -19,21 +19,25 @@
> [% INCLUDE 'masthead.inc' %]
> 
>     <div class="main">
>-        <nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumbs">
>-            <ol class="breadcrumb">
>-                <li class="breadcrumb-item">
>-                    <a href="/cgi-bin/koha/opac-main.pl">Home</a>
>-                </li>
>-                [% IF ( logged_in_user ) %]
>-                    <li class="breadcrumb-item">
>-                        <a href="/cgi-bin/koha/opac-user.pl">[% INCLUDE 'patron-title.inc' patron = logged_in_user %]</a>
>-                    </li>
>+        [% WRAPPER breadcrumbs %]
>+            [% IF ( logged_in_user ) %]
>+                [% WRAPPER breadcrumb_item %]
>+                    <a href="/cgi-bin/koha/opac-user.pl">[% INCLUDE 'patron-title.inc' patron = logged_in_user %]</a>
>                 [% END %]
>-                <li class="breadcrumb-item active">
>-                    <a href="#" aria-current="page">Purchase suggestions</a>
>-                </li>
>-            </ol>
>-        </nav> 
>+            [% END %]
>+            [% IF ( op_add ) %]
>+                [% WRAPPER breadcrumb_item %]
>+                    <a href="/cgi-bin/koha/opac-suggestions.pl">Purchase suggestions</a>
>+                [% END %]
>+                [% WRAPPER breadcrumb_item bc_active= 1 %]
>+                    <span>New purchase suggestion</span>
>+                [% END %]
>+            [% ELSE %]
>+                [% WRAPPER breadcrumb_item bc_active= 1 %]
>+                    <span>Purchase suggestions</span>
>+                [% END %]
>+            [% END %]
>+        [% END #/ WRAPPER breadcrumbs %]
> 
>         <div class="container-fluid">
>             <div class="row">
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-tags_subject.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-tags_subject.tt
>index a357a5712f..d241fdb3b1 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-tags_subject.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-tags_subject.tt
>@@ -1,3 +1,4 @@
>+[% USE raw %]
> [% USE Koha %]
> [% USE AdditionalContents %]
> [% SET OpacNav = AdditionalContents.get( location => "OpacNav", lang => lang, library => logged_in_user.branchcode || default_branch, blocktitle => 0 ) %]
>@@ -11,16 +12,11 @@
> [% INCLUDE 'masthead.inc' %]
> 
>     <div class="main">
>-        <nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumbs">
>-            <ol class="breadcrumb">
>-                <li class="breadcrumb-item">
>-                    <a href="/cgi-bin/koha/opac-main.pl">Home</a>
>-                </li>
>-                <li class="breadcrumb-item active">
>-                    <a href="#" aria-current="page">Subject cloud</a>
>-                </li>
>-            </ol>
>-        </nav> 
>+        [% WRAPPER breadcrumbs %]
>+            [% WRAPPER breadcrumb_item bc_active= 1 %]
>+                <span>Subject cloud</span>
>+            [% END %]
>+        [% END #/ WRAPPER breadcrumbs %]
> 
>         <div class="container-fluid">
>             <div class="row">
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-topissues.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-topissues.tt
>index f6ec596210..7571a0fdb5 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-topissues.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-topissues.tt
>@@ -1,3 +1,4 @@
>+[% USE raw %]
> [% USE Koha %]
> [% USE Branches %]
> [% USE AuthorisedValues %]
>@@ -41,16 +42,11 @@
> [% INCLUDE 'masthead.inc' %]
> 
>     <div class="main">
>-        <nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumbs">
>-            <ol class="breadcrumb">
>-                <li class="breadcrumb-item">
>-                    <a href="/cgi-bin/koha/opac-main.pl">Home</a>
>-                </li>
>-                <li class="breadcrumb-item active">
>-                    <a href="#" aria-current="page">Most popular titles</a>
>-                </li>
>-            </ol>
>-        </nav> 
>+        [% WRAPPER breadcrumbs %]
>+            [% WRAPPER breadcrumb_item bc_active= 1 %]
>+                <span>Most popular titles</span>
>+            [% END %]
>+        [% END #/ WRAPPER breadcrumbs %]
> 
>         <div class="container-fluid">
>             <div class="row">
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/svc/suggestion.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/svc/suggestion.tt
>index aab088ff2d..2c0a7180da 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/svc/suggestion.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/svc/suggestion.tt
>@@ -1,3 +1,4 @@
>+[% USE raw %]
> [% USE Koha %]
> [% USE AdditionalContents %]
> [% SET OpacNav = AdditionalContents.get( location => "OpacNav", lang => lang, library => logged_in_user.branchcode || default_branch, blocktitle => 0 ) %]
>@@ -12,16 +13,11 @@
>     [% INCLUDE 'masthead.inc' %]
> 
>     <div class="main">
>-        <nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumbs">
>-            <ol class="breadcrumb">
>-                <li class="breadcrumb-item">
>-                    <a href="/cgi-bin/koha/opac-main.pl">Home</a>
>-                </li>
>-                <li class="breadcrumb-item active">
>-                    <a href="#" aria-current="page">Search suggestions</a>
>-                </li>
>-            </ol>
>-        </nav> 
>+        [% WRAPPER breadcrumbs %]
>+            [% WRAPPER breadcrumb_item bc_active= 1 %]
>+                <span>Search suggestions</span>
>+            [% END %]
>+        [% END #/ WRAPPER breadcrumbs %]
> 
>         <div class="container-fluid">
>             <div class="row">
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 34866: 156004 | 158239 | 158265 | 158384