Bug 34307: (follow-up) Use template wrapper for title tag
This patch updates the title tag to use a wrapper too, since Bug 33906 is in the process of getting fixed. The patch also removes an extra <span> tag: Only the translatable string needs the span, not the entire line. Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
parent
a6ec493647
commit
62a757acd1
1 changed files with 14 additions and 11 deletions
|
@ -1,19 +1,22 @@
|
|||
[% USE raw %]
|
||||
[% PROCESS 'i18n.inc' %]
|
||||
[% SET footerjs = 1 %]
|
||||
[% INCLUDE 'doc-head-open.inc' %]
|
||||
[% IF ( ! method ) %]
|
||||
[% method = 'tool'; %]
|
||||
[% method = 'tool'; %]
|
||||
[% END %]
|
||||
<title>
|
||||
[% IF ( plugin_title ) %][% plugin_title | html %] › [% END %]
|
||||
Plugins ›
|
||||
[% IF ( method == 'tool' ) %]
|
||||
Tools
|
||||
[% ELSE %]
|
||||
Administration
|
||||
<title>[% FILTER collapse %]
|
||||
[% IF ( plugin_title ) %]
|
||||
[% plugin_title | html %] ›
|
||||
[% END %]
|
||||
› Koha
|
||||
</title>
|
||||
[% t("Plugins") | html %] ›
|
||||
[% IF ( method == 'tool' ) %]
|
||||
[% t("Tools") | html %]
|
||||
[% ELSE %]
|
||||
[% t("Administration") | html %]
|
||||
[% END %] ›
|
||||
[% t("Koha") | html %]
|
||||
[% END %]</title>
|
||||
[% INCLUDE 'doc-head-close.inc' %]
|
||||
</head>
|
||||
|
||||
|
@ -35,7 +38,7 @@
|
|||
<a href="/cgi-bin/koha/plugins/plugins-home.pl">Plugins</a>
|
||||
[% END %]
|
||||
[% WRAPPER breadcrumb_item bc_active= 1 %]
|
||||
<span>[% IF ( plugin_title ) %][% plugin_title | html %][% ELSE %]<span>Plugin</span>[% END %]</span>
|
||||
[% IF ( plugin_title ) %][% plugin_title | html %][% ELSE %]<span>Plugin</span>[% END %]
|
||||
[% END %]
|
||||
[% END #/ WRAPPER breadcrumbs %]
|
||||
[% END #/ WRAPPER sub-header.inc %]
|
||||
|
|
Loading…
Reference in a new issue