From 064ecc7bfe53a960bd899e7352327c1182573782 Mon Sep 17 00:00:00 2001
From: Aspectis <tobias@aspectis.net>
Date: Tue, 16 Feb 2021 23:43:01 +0100
Subject: [PATCH] refs #19054 [fid_bbi] redesign record access section
* Move links to the top
* Move charges notice to the bottom
* Remove duplicate links
* Remove redundant "Online Links" caption
* Remove `target="_blank"` from external links; it should be the user's decision if links are opened in the current tab or a new one
* Remove breaks from strings
* On small screens:
** Remove the vertical decorative lines to make better use of available space
** Change tabs layout so all tabs are clickable, highlight the current tab with a connecting line
* fix label on free resources
** Freely available resources no longer have contradictory labels.
** Fix typo in translations.
* increase margin on WorldCat link list
* improve get-it box
** Add external-link icon
** Use external-link view helper
** Update translations: Remove breaks and fix capitalization
* fix resolver links (WIP)
** Remove duplicate resolver links
** Style print links
* Fix margins
THIS IS A WORK IN PROGRESS AND REQUIRES FURTHER CHANGES.
---
fid_bbi/languages/Facet/de.ini | 4 +-
fid_bbi/languages/de.ini | 22 +-
fid_bbi/languages/en.ini | 20 +-
themes/fid_bbi/icons/small/external-link.svg | 1 +
themes/fid_bbi/languages/de.ini | 4 +
themes/fid_bbi/languages/en.ini | 4 +
themes/fid_bbi/scss/blocks/record.scss | 72 ++++-
themes/fid_bbi/scss/blocks/tabs.scss | 62 +++-
themes/fid_bbi/scss/util/hacks.scss | 27 +-
.../RecordDriver/DefaultRecord/core.phtml | 11 +-
.../templates/RecordTab/holdingsils.phtml | 300 +++++++++---------
.../fid_bbi/templates/ajax/boss-false.phtml | 33 ++
themes/fid_bbi/templates/ajax/boss-true.phtml | 29 ++
themes/fid_bbi/templates/ajax/boss.phtml | 4 +
.../templates/ajax/resolverLinks-print.phtml | 95 ++++++
.../templates/ajax/resolverLinks.phtml | 96 +++---
.../fid_bbi/templates/get-it-box/links.phtml | 90 +++---
17 files changed, 565 insertions(+), 309 deletions(-)
create mode 100644 themes/fid_bbi/icons/small/external-link.svg
create mode 100644 themes/fid_bbi/templates/ajax/boss-false.phtml
create mode 100644 themes/fid_bbi/templates/ajax/boss-true.phtml
create mode 100644 themes/fid_bbi/templates/ajax/boss.phtml
create mode 100644 themes/fid_bbi/templates/ajax/resolverLinks-print.phtml
diff --git a/fid_bbi/languages/Facet/de.ini b/fid_bbi/languages/Facet/de.ini
index 0ba13336976..6414016bdb1 100644
--- a/fid_bbi/languages/Facet/de.ini
+++ b/fid_bbi/languages/Facet/de.ini
@@ -20,7 +20,7 @@ facet_avail = "Zugang"
Availability = Zugang
Online = Online-Ressourcen
#16903
-Free = Kostenfrei Zugänglich
+Free = Kostenfrei zugänglich
Local = Lokale Bestände
Non-Free = Beschränkter Zugang
@@ -34,4 +34,4 @@ Non-Free = Beschränkter Zugang
183 = "K10plus"
188 = "GoeScholar - Publikationenserver der Georg-August-Universität Göttingen"
-Digitale Sammlungen UBL = "Buchhandschriften der UB Leipzig"
\ No newline at end of file
+Digitale Sammlungen UBL = "Buchhandschriften der UB Leipzig"
diff --git a/fid_bbi/languages/de.ini b/fid_bbi/languages/de.ini
index a6d9c18ded1..3301f40af18 100644
--- a/fid_bbi/languages/de.ini
+++ b/fid_bbi/languages/de.ini
@@ -417,7 +417,7 @@ Datensammlung = Datenquelle
browse_sources = "Datenquellen durchsuchen"
-charges_may_apply = "Eventuell kostenpflichtig"
+charges_may_apply = "Eventuell kostenpflichtig."
#12932
embed_searchbox_button = "Suche einbinden"
@@ -465,7 +465,7 @@ Be the first to tag this record = "Fügen Sie einen Tag hinzu"
add_tag_success = "Tags gespeichert. Beachten Sie, dass Tags nach einer Woche öffentlich sichtbar werden."
#16903
-Open Access = Kostenfrei Zugänglich
+Open Access = Kostenfrei zugänglich
#16470
Link to WorldCat record = "Link zum Datensatz des WorldCat"
@@ -481,18 +481,18 @@ getit_order = "Kostenlos bestellen"
getit_partcopy = "Teilkopie besorgen"
getit_partcopy_notice = "(max. 10 % des Gesamtumfangs)"
getit_text_1 = "Diese Ressource ist frei verfügbar."
-getit_text_2 = "Diese Neuerscheinung können Sie zur Leihe bestellen, wenn Sie angemeldet sind.<br/><br/>Noch keinen Account? %s"
-getit_text_3_1 = "Sie müssen angemeldet sein, um Zugang zu diesem Titel zu erhalten.<br /><br />Noch keinen Account? %s"
-getit_text_3_2 = "Jetzt registrieren"
+getit_text_2 = "Diese Neuerscheinung können Sie zur Leihe bestellen, wenn Sie angemeldet sind. Noch keinen Account? %s"
+getit_text_3_1 = "Sie müssen angemeldet sein, um Zugang zu diesem Titel zu erhalten. Noch keinen Account? %s"
+getit_text_3_2 = "Jetzt registrieren."
getit_text_4 = "Ein Teil der Inhalte ist ggf. Nicht kostenlos als Volltext zugänglich."
-getit_text_5 = "Dieser Inhalt wird zur Verfügung gestellt über eine Nationallizenz, gefördert durch die DFG.<br/>Ggf. Ist eine Registrierung auf www.nationallizenzen.de notwendig."
+getit_text_5 = "Dieser Inhalt wird zur Verfügung gestellt über eine Nationallizenz, gefördert durch die DFG. Ggf. Ist eine Registrierung auf www.nationallizenzen.de notwendig."
getit_text_6 = "Zugang zum Digitalisat erhalten Sie bei einigen Titeln dieser Quelle über den angezeigten Link."
-getit_text_7 = "Dieser Titel stammt aus der Bielefeld Academic Search Engine.<br/><br/>Einige Dokumente aus dieser Quelle sind möglicherweise nicht als Volltext zugänglich."
-getit_text_8 = "Für diesen Titel können wir derzeit leider keine weiteren Informationen zur Verfügbarkeit bereitstellen.<br/><br/>Ob der Titel per Fernleihe bestellbar ist, kann Ihnen Ihre Heimatbibliothek mitteilen. Einige Medientypen (wie z. B. Manuskripte, Schallplatten, CDs, Videos) können von der Fernleihe ausgeschlossen sein."
+getit_text_7 = "Dieser Titel stammt aus der Bielefeld Academic Search Engine. Einige Dokumente aus dieser Quelle sind möglicherweise nicht als Volltext zugänglich."
+getit_text_8 = "Für diesen Titel können wir derzeit leider keine weiteren Informationen zur Verfügbarkeit bereitstellen. Ob der Titel per Fernleihe bestellbar ist, kann Ihnen Ihre Heimatbibliothek mitteilen. Einige Medientypen (wie z. B. Manuskripte, Schallplatten, CDs, Videos) können von der Fernleihe ausgeschlossen sein."
getit_text_9 = "Online-Ressourcen können Lizenzen erfordern oder kostenpflichtig sein."
-getit_text_10 = "Weitere Informationen sehen Sie, wenn Sie angemeldet sind.<br /><br />Noch keinen Account? %s"
+getit_text_10 = "Weitere Informationen sehen Sie, wenn Sie angemeldet sind. Noch keinen Account? %s"
getit_text_11 = "Dies ist eine Gesamtaufnahme einer Zeitschrift oder einer mehrteiligen Monographie. Verknüpfte Einzeltitel sind unter „Bände“ aufgeführt."
-getit_text_12 = "Bestellen Sie ein Leihexemplar dieser Neuerscheinung auf Kosten des FID BBI. Das Buch wird direkt an Sie geschickt.<br/><br/>"
+getit_text_12 = "Bestellen Sie ein Leihexemplar dieser Neuerscheinung auf Kosten des FID BBI. Das Buch wird direkt an Sie geschickt."
getit_text_13 = "Dieses E-Book ist für Sie über eine FID BBI-Lizenz freigeschaltet."
getit_text_14 = "Einige Medientypen (wie z. B. Manuskripte, Schallplatten, CDs, Videos) können von der Fernleihe ausgeschlossen sein."
getit_text_15 = "Wenn dieses E-Book bei Ihrer Heimatbibliothek als vorhanden gekennzeichnet ist, sollte der direkte Zugang über den obigen Link ohne weiteres möglich sein."
@@ -511,4 +511,4 @@ ezb_period = "Jahrgänge"
ezb_comment = "Hinweise"
#17038
-level_homepage = Link zur Homepage
\ No newline at end of file
+level_homepage = Link zur Homepage
diff --git a/fid_bbi/languages/en.ini b/fid_bbi/languages/en.ini
index eaeeb3beb72..f5771e7129b 100644
--- a/fid_bbi/languages/en.ini
+++ b/fid_bbi/languages/en.ini
@@ -406,7 +406,7 @@ Datensammlung = Data Collection
browse_sources = "Browse Data Collections"
-charges_may_apply = "Charges may apply"
+charges_may_apply = "Charges may apply."
#12932
embed_searchbox_button = "Embed search"
@@ -473,18 +473,18 @@ getit_order = "Order Free of Charge"
getit_partcopy = "Get part copy"
getit_partcopy_notice = "(max. 10 % of the entire book)"
getit_text_1 = "This resource is open access."
-getit_text_2 = "You can order this new publication on loan when you are logged in.<br/><br/>No account yet? %s"
-getit_text_3_1 = "You will see more information when you are logged in.<br/><br/>No account yet? %s"
-getit_text_3_2 = "Register now"
-getit_text_4 = "This title is recorded in the IZI database of the International Central Institute for Youth and Educational Television.<br/><br/>Access to the full text is only available for some titles of this source via the link above."
-getit_text_5 = "This content is provided under a national license, funded by the DFG.<br/>It may be necessary to register at www.nationallizenzen.de."
+getit_text_2 = "You can order this new publication on loan when you are logged in. No account yet? %s"
+getit_text_3_1 = "You will see more information when you are logged in. No account yet? %s"
+getit_text_3_2 = "Register now."
+getit_text_4 = "This title is recorded in the IZI database of the International Central Institute for Youth and Educational Television. Access to the full text is only available for some titles of this source via the link above."
+getit_text_5 = "This content is provided under a national license, funded by the DFG. It may be necessary to register at www.nationallizenzen.de."
getit_text_6 = "You can access the digitised version of some titles of this source via the link shown."
-getit_text_7 = "This title is from the Bielefeld Academic Search Engine.<br/><br/>Some documents from this source may not be accessible as full text."
-getit_text_8 = "Unfortunately, we cannot provide any further information on availability for this title at the moment.<br/><br/>Your home library can tell you whether the title can be ordered by interlibrary loan. Some media types (e.g. manuscripts, records, CDs, videos) may be excluded from interlibrary loan."
+getit_text_7 = "This title is from the Bielefeld Academic Search Engine. Some documents from this source may not be accessible as full text."
+getit_text_8 = "Unfortunately, we cannot provide any further information on availability for this title at the moment. Your home library can tell you whether the title can be ordered by interlibrary loan. Some media types (e.g. manuscripts, records, CDs, videos) may be excluded from interlibrary loan."
getit_text_9 = "Access to electronic resources may be subject to a license or a fee."
-getit_text_10 = "You will see more information when you are logged in.<br/><br/>No account yet? %s"
+getit_text_10 = "You will see more information when you are logged in. No account yet? %s"
getit_text_11 = "This is a comprehensive entry of a journal or a multipart monograph. Linked individual titles are listed under "Volumes"."
-getit_text_12 = "Order a loan copy of this new publication at the expense of FID BBI. The book will be sent directly to you.<br/><br/>"
+getit_text_12 = "Order a loan copy of this new publication at the expense of FID BBI. The book will be sent directly to you."
getit_text_13 = "This e-book is free to access for you via an FID BBI license."
getit_text_14 = "Some media types (e.g. manuscripts, records, CDs, videos) may be excluded from interlibrary loan."
getit_text_15 = "If this e-book is marked as available in your home library, you should be able to access it directly via the link above."
diff --git a/themes/fid_bbi/icons/small/external-link.svg b/themes/fid_bbi/icons/small/external-link.svg
new file mode 100644
index 00000000000..a5a234d8350
--- /dev/null
+++ b/themes/fid_bbi/icons/small/external-link.svg
@@ -0,0 +1 @@
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 28 28" height="28" width="28"><path d="M14 7H7v14h14v-7M16 5h7v7M23 5l-9 9" fill="none"/></svg>
diff --git a/themes/fid_bbi/languages/de.ini b/themes/fid_bbi/languages/de.ini
index afe496046d7..2dfa9bcc701 100644
--- a/themes/fid_bbi/languages/de.ini
+++ b/themes/fid_bbi/languages/de.ini
@@ -35,6 +35,10 @@ the year = zum Jahr
This field is required = Pflichtfeld
to = bis
+boss_true = In Ihrer Bibliothek vorhanden
+boss_false = Nicht in Ihrer Heimatbibliothek vorhanden. Ob der Titel per Fernleihe bestellbar ist, kann Ihnen Ihre Heimatbibliothek mitteilen.
+boss_noISXNZBD = "Für diesen Titel können wir derzeit leider keine weiteren Informationen zur Verfügbarkeit bereitstellen. Ob der Titel per Fernleihe bestellbar ist, kann Ihnen Ihre Heimatbibliothek mitteilen."
+
history_purge = Nicht gespeicherte Suchanfragen entfernen
history_recent_searches = Ihre aktuellen Suchanfragen
diff --git a/themes/fid_bbi/languages/en.ini b/themes/fid_bbi/languages/en.ini
index 5e1704ec530..f19989e3a0b 100644
--- a/themes/fid_bbi/languages/en.ini
+++ b/themes/fid_bbi/languages/en.ini
@@ -12,6 +12,10 @@ Scroll to top = Scroll to top
Team = Team
This field is required = This field is required
+boss_true = Available in your library
+boss_false = "Not available in your home library. Your home library can tell you whether the title can be ordered by interlibrary loan."
+boss_noISXNZBD = "Unfortunately, we cannot provide any further information on availability for this title at the moment. Your home library can tell you whether the title can be ordered by interlibrary loan."
+
history_purge = Discard unsaved searches
history_recent_searches = Your Recent Searches
diff --git a/themes/fid_bbi/scss/blocks/record.scss b/themes/fid_bbi/scss/blocks/record.scss
index beff50f9c21..40502468a24 100644
--- a/themes/fid_bbi/scss/blocks/record.scss
+++ b/themes/fid_bbi/scss/blocks/record.scss
@@ -17,6 +17,11 @@
@include heading(3);
}
+ h3 {
+ font: inherit;
+ margin: g() 0 g(.5);
+ }
+
.box {
padding: g() g() g(1 + 1);
@@ -28,22 +33,44 @@
}
.record_access {
- border-left: 2px solid;
display: block;
- padding-left: g() - 2px;
@media #{$bp3} {
display: flex;
}
+ @media #{$bp5} {
+ border-left: 2px solid;
+ padding-left: g() - 2px;
+ }
+
h2 {
color: inherit;
- flex: 0 0 g(8 - 1.25); // Space for link icons
- margin: 0;
+ flex: 0 0 g(8);
+
+ @media #{$bp3} {
+ margin: 0;
+ }
}
- ul {
- margin: 0;
+ p {
+ margin: 0 0 g(.5);
+ }
+
+ .link-with-icon {
+ margin-left: 0; // unindent icons
+ }
+
+ .unstyled-list {
+ margin-bottom: g(.5);
+
+ &:last-child {
+ margin-bottom: 0;
+ }
+
+ li + li {
+ margin-top: 0;
+ }
}
}
@@ -88,14 +115,25 @@
width: 100%;
}
+.record_getitbox {
+ .boss-data-false,
+ .boss-data-true {
+ &:empty {
+ display: none;
+ }
+ }
+}
+
.record_header {
flex: 1;
> h1,
> dl {
- border-left: 2px solid $text-color;
- margin-left: 0;
- padding-left: g() - 2px;
+ @media #{$bp5} {
+ border-left: 2px solid $text-color;
+ margin-left: 0;
+ padding-left: g() - 2px;
+ }
}
> h1 {
@@ -103,6 +141,14 @@
}
}
+.record_holdingsils {
+ .ezb-message {
+ &:not(:last-child) {
+ margin-bottom: g(.5);
+ }
+ }
+}
+
.record table,
.record_internal {
margin: 0 0 g();
@@ -150,8 +196,12 @@
}
dl {
- border-left: 2px solid;
- padding-left: g() - 2px;
+ max-width: none;
+
+ @media #{$bp5} {
+ border-left: 2px solid;
+ padding-left: g() - 2px;
+ }
}
}
diff --git a/themes/fid_bbi/scss/blocks/tabs.scss b/themes/fid_bbi/scss/blocks/tabs.scss
index 44120737e1b..ebcfebe54f2 100644
--- a/themes/fid_bbi/scss/blocks/tabs.scss
+++ b/themes/fid_bbi/scss/blocks/tabs.scss
@@ -21,30 +21,72 @@
.record-tab,
.tabs_button {
+ position: relative;
+ padding-left: g(1.25);
+
+ @media #{$bp2} {
+ padding-left: 0;
+ }
+
a {
- @include button;
- border-color: transparent;
- display: block;
+ @media #{$bp2} {
+ @include button;
+ border-color: transparent;
+ display: block;
+ margin-bottom: 0;
+ }
}
- &.active a {
- background: #fff;
- border-bottom: 0;
- border-color: $text-color;
- color: $text-color;
- padding-bottom: g(.5);
+ &.active {
+ &::after {
+ content: '';
+ display: block;
+ position: absolute;
+ width: g(.5);
+ height: 99em;
+ border-top: 2px solid;
+ border-left: 2px solid;
+ top: 50%;
+ left: g(.5);
+
+ @media #{$bp2} {
+ content: none;
+ }
+ }
+
+ a {
+ color: $text-color;
+ box-shadow: none;
+
+ @media #{$bp2} {
+ background: #fff;
+ border-bottom: 0;
+ border-color: $text-color;
+ padding-bottom: g(.5);
+ }
+ }
}
}
.tabs_header {
- display: flex;
margin: 0;
+ overflow: hidden;
padding: 0;
max-width: none;
position: relative;
z-index: 1;
+ @media #{$bp2} {
+ display: flex;
+ }
+
li {
+ margin-bottom: g(.5);
+
+ @media #{$bp2} {
+ margin-bottom: 0;
+ }
+
&::before {
content: none; // remove bullet
}
diff --git a/themes/fid_bbi/scss/util/hacks.scss b/themes/fid_bbi/scss/util/hacks.scss
index cc1da372007..401440ebf73 100644
--- a/themes/fid_bbi/scss/util/hacks.scss
+++ b/themes/fid_bbi/scss/util/hacks.scss
@@ -40,7 +40,7 @@ a.active {
display: none;
list-style: none;
margin-top: g(.25);
- padding: g(.5) g();
+ padding: g(.5) g(.75);
position: absolute;
z-index: 9;
@@ -153,3 +153,28 @@ a.active {
margin-top: 0;
}
}
+
+.get-it-box-links {
+ margin-bottom: g(.5);
+
+ &:last-child {
+ margin-bottom: 0;
+ }
+}
+
+.openurls {
+ dd {
+ margin: 0;
+ }
+
+ dt {
+ @media #{$bp3} {
+ width: g(4);
+ margin: 0;
+ }
+ }
+}
+
+.resolver-links {
+ margin-bottom: g(.5);
+}
diff --git a/themes/fid_bbi/templates/RecordDriver/DefaultRecord/core.phtml b/themes/fid_bbi/templates/RecordDriver/DefaultRecord/core.phtml
index 8eb95b0b499..4709622094e 100644
--- a/themes/fid_bbi/templates/RecordDriver/DefaultRecord/core.phtml
+++ b/themes/fid_bbi/templates/RecordDriver/DefaultRecord/core.phtml
@@ -63,14 +63,9 @@
<div class="box -left">
<div class="record_access">
<h2><?=$this->translate('Access')?></h2>
- <ul class="unstyled-list">
- <li>
- <a class="link-with-icon" href="#TODO">
- <?=$this->icon('small/arrow-right')?>
- <?=$this->context($this)->renderInContext('RecordTab/holdingsils.phtml',[])?>
- </a>
- </li>
- </ul>
+ <div class="record_holdingsils">
+ <?=$this->context($this)->renderInContext('RecordTab/holdingsils.phtml',[])?>
+ </div>
</div>
</div>
diff --git a/themes/fid_bbi/templates/RecordTab/holdingsils.phtml b/themes/fid_bbi/templates/RecordTab/holdingsils.phtml
index 033ede81dae..2d258af1eb4 100644
--- a/themes/fid_bbi/templates/RecordTab/holdingsils.phtml
+++ b/themes/fid_bbi/templates/RecordTab/holdingsils.phtml
@@ -1,163 +1,167 @@
<!-- fid_bbi: recordTab - holdingsils -->
<?php
- // Set up convenience variables:
- $account = $this->auth()->getManager();
- $user = $account->isLoggedIn();
- $openUrl = $this->openUrl($this->driver, 'holdings');
- $openUrlActive = $openUrl->isActive();
- $doi = $this->doi($this->driver, 'holdings');
- $doiActive = $doi->isActive();
- // Account for replace_other_urls setting
- $urls = $this->record($this->driver)->getLinkDetails($openUrlActive);
- // #15930 Get It Box
- $getitConfig = $this->getit($this->driver)->getConfig();
- $id = $this->driver->getUniqueId();
- // finc-specific: externalCatalogueLink
- $extUrls = $this->externalCatalogueLink($this->driver)->getLinks();
- // Set page title.
- $this->headTitle($this->translate('Holdings') . ': ' . $this->driver->getBreadcrumb());
+// Set up convenience variables:
+$account = $this->auth()->getManager();
+$user = $account->isLoggedIn();
+$openUrl = $this->openUrl($this->driver, 'holdings');
+$openUrlActive = $openUrl->isActive();
+$doi = $this->doi($this->driver, 'holdings');
+$doiActive = $doi->isActive();
+// Account for replace_other_urls setting
+$urls = $this->record($this->driver)->getLinkDetails($openUrlActive);
+// #15930 Get It Box
+$getitConfig = $this->getit($this->driver)->getConfig();
+$id = $this->driver->getUniqueId();
+// finc-specific: externalCatalogueLink
+$extUrls = $this->externalCatalogueLink($this->driver)->getLinks();
+
+$this->headTitle($this->translate('Holdings') . ': ' . $this->driver->getBreadcrumb());
+
+$isFree = in_array('Free', $this->driver->tryMethod('getFacetAvail'));
?>
-<table class="table table-striped">
- <?php /* finc: add 'Online Access' in holdings-tab #13770 - VE */ ?>
- <?php if (!empty($urls)): ?>
- <?php if (in_array('Free',$this->driver->tryMethod('getFacetAvail'))):?>
- <caption><?=$this->translate('Open Access')?></caption>
- <?php elseif (!empty(array_filter($urls,
- function ($elem) {
- return !isset($elem['desc']) || strpos($elem['desc'],'Volltext') !== false;
- }
- ))):?>
- <caption><?=$this->translate('charges_may_apply')?></caption>
- <?php else: ?>
- <caption><?=$this->translate('online_availability')?></caption>
- <?php endif;?>
- <tr>
- <th><?=$this->transEsc("Link")?>:</th>
- <td>
- <?php foreach ($urls as $current): ?>
- <a href="<?=$this->escapeHtmlAttr($this->proxyUrl($current['url']))?>" target="_blank"><?=$this->escapeHtml($current['desc'])?></a><br/>
- <?php endforeach; ?>
- </td>
- </tr>
- <?php endif; ?>
- <?php /* Other Issues */ ?>
- <?php
- $onlineUrls = $this->record($this->driver)->getOnlineEditions();
- if (!empty($onlineUrls)): ?>
- <tr>
- <th><?=$this->transEsc("Other Editions")?>:</th>
- <td>
- <?php foreach ($onlineUrls as $current): ?>
- <a href="<?=$this->escapeHtmlAttr($this->proxyUrl($current['link']))?>" target="_blank"><?=$this->escapeHtml($current['text'])?></a><br/>
- <?php endforeach; ?>
- </td>
- </tr>
- <?php endif; ?>
- <? /* finc-specific snippet - #9274 - replaces if ($openUrlActive): - adapted for de_15 - CK */ ?>
- <? if ($openUrlActive): ?>
- <?=$openUrl->renderTemplate()?>
- <? endif; ?>
- <?php /* Availability */ ?>
- <?php if ($user && $getitConfig['bossData']): ?>
- <? $script = <<<JS
- $(document).ready(function() {
- var recordId = $('.hiddenId').val();
- var recordSource = $('.hiddenSource').val();
- $.ajax({
- dataType: 'json',
- url: VuFind.path + '/AJAX/JSON?method=getBoss',
- method: 'GET',
- data: {id: recordId, source: recordSource}
- }).done(function(response) {
- if (response.data.isISIL) {
- $('.boss-data-true').html(response.data.html);
- } else {
- $('.boss-data-false').html(response.data.html);
- $('.boss-notice').addClass('hidden');
- }
- // Use to register new lightbox window networknotice
- VuFind.lightbox.bind();
- });
- });
-JS;
- ?>
- <?=$this->inlineScript(\Zend\View\Helper\HeadScript::SCRIPT, $script, 'SET');?>
- <?php endif; ?>
- <?php if ($getitConfig['hideNotice']): ?>
- <? $script = <<<JS
- $(document).ready(function() {
- $('.getitbox-notices').addClass('hidden');
- });
-JS;
- ?>
- <?=$this->inlineScript(\Zend\View\Helper\HeadScript::SCRIPT, $script, 'SET');?>
- <?php endif; ?>
+<?php // finc: add 'Online Access' in holdings-tab #13770 - VE ?>
+<?php if (!empty($urls)): ?>
+ <?php if ($isFree): ?>
+ <p><?=$this->translate('Open Access')?></p>
+ <?php endif; ?>
+<?php endif; ?>
+
+<div class="record_getitbox">
+ <?php if ($user && $getitConfig['bossData']): ?>
+ <p class="boss-data-true"></p>
+ <?php endif; ?>
- <div class="getitbox-getit">
- <h4><?=$getitConfig['boxHeadline']?></h4>
+ <?php if ($openUrlActive && $getitConfig['showArticleButton']): ?>
+ <div class="openUrlButton"></div>
+ <?php endif; ?>
- <?//=($offlineMode == "ils-offline") ? $this->render('Helpers/ils-offline.phtml', ['offlineModeMsg' => 'ils_offline_holdings_message']) : ''?>
+ <?php if ($getitConfig['showOrderButton'] && $user): ?>
+ <p>
+ <a
+ data-lightbox
+ href="<?=$this->url('record-fidpda', array('id' => $id))?>" rel="nofollow"
+ >
+ <?=$this->transEsc('getit_order')?>
+ </a>
+ </p>
+ <?php endif; ?>
- <?php if ($user && $getitConfig['bossData']): ?>
- <div class="boss-data-true"></div>
- <?php endif; ?>
+ <?php if ($getitConfig['showPartCopyButton'] && $user): ?>
+ <p>
+ <a
+ data-lightbox
+ href="<?=$this->url('record-fidsubitopartialcopy', array('id' => $id))?>" rel="nofollow"
+ >
+ <?=$this->transEsc('getit_partcopy')?>
+ </a>
+ </p>
+ <div class="pda-part-text">
+ <?=$this->transEsc('getit_partcopy_notice')?>
+ </div>
+ <?php endif; ?>
- <?php if ($openUrlActive && $getitConfig['showArticleButton']): ?>
- <div class="openUrlButton"></div>
- <?php endif; ?>
- <?php if ($getitConfig['showOrderButton'] && $user): ?>
- <a class="btn btn-primary pda-button full-order" data-lightbox href="<?=$this->url('record-fidpda', array('id' => $id))?>" rel="nofollow">
- <?=$this->transEsc('getit_order')?>
+ <?php // finc: add 'Online Access' in holdings-tab #13770 - VE ?>
+ <?php if ($getitConfig['showLinks'] && (!empty($urls) || $openUrlActive || $doiActive)): ?>
+ <?php $linksArray = compact('getitConfig', 'openUrl', 'openUrlActive', 'urls', 'doiActive', 'doi'); ?>
+ <?=$this->context($this)->renderInContext('get-it-box/links.phtml', $linksArray)?>
+ <?php endif; ?>
+
+ <?php if (!$isFree && !empty(array_filter($urls,
+ function ($elem) {
+ return !isset($elem['desc']) || strpos($elem['desc'], 'Volltext') !== false;
+ }
+ ))): ?>
+ <p>
+ <?=$this->translate('charges_may_apply')?>
+ </p>
+ <?php endif;?>
+</div>
+
+<?php // Other Issues ?>
+<?php $onlineUrls = $this->record($this->driver)->getOnlineEditions(); ?>
+<?php if (!empty($onlineUrls)): ?>
+ <h3><?=$this->transEsc('Other Editions')?></h3>
+ <ul class="unstyled-list">
+ <?php foreach ($onlineUrls as $current): ?>
+ <li>
+ <a class="link-with-icon" href="<?=$this->escapeHtmlAttr($this->proxyUrl($current['link']))?>">
+ <?=$this->icon('small/arrow-right')?>
+ <?=$this->escapeHtml($current['text'])?>
</a>
- <?php endif; ?>
-
- <?php if ($getitConfig['showPartCopyButton'] && $user): ?>
- <a class="btn btn-primary pda-button part-order" data-lightbox href="<?=$this->url('record-fidsubitopartialcopy', array('id' => $id))?>" rel="nofollow">
- <?=$this->transEsc('getit_partcopy')?>
- </a><br />
- <div class="pda-part-text">
- <?=$this->transEsc('getit_partcopy_notice')?>
- </div>
- <?php endif; ?>
-
- <?php /* finc: add 'Online Access' in holdings-tab #13770 - VE */ ?>
- <?php if ($getitConfig['showLinks'] && (!empty($urls) || $openUrlActive || $doiActive)): ?>
- <?= $this->context($this)->renderInContext('get-it-box/links.phtml',
- compact('getitConfig', 'openUrl','openUrlActive','urls','doiActive','doi')
- ); ?>
- <?php endif; ?>
+ </li>
+ <?php endforeach; ?>
+ </ul>
+<?php endif; ?>
+
+<?php // finc-specific snippet - #9274 - replaces if ($openUrlActive): - adapted for de_15 - CK ?>
+<?php if ($openUrlActive): ?>
+ <?=$openUrl->renderTemplate()?>
+<?php endif; ?>
+
+<?php // Availability ?>
+<?php if ($user && $getitConfig['bossData']): ?>
+ <?php $script = <<<JS
+$(document).ready(function() {
+ var recordId = $('.hiddenId').val();
+ var recordSource = $('.hiddenSource').val();
+ $.ajax({
+ dataType: 'json',
+ url: VuFind.path + '/AJAX/JSON?method=getBoss',
+ method: 'GET',
+ data: {id: recordId, source: recordSource}
+ }).done(function(response) {
+ if (response.data.isISIL) {
+ $('.boss-data-true').html(response.data.html);
+ } else {
+ $('.boss-data-false').html(response.data.html);
+ $('.boss-notice').addClass('hidden');
+ }
+ // Use to register new lightbox window networknotice
+ VuFind.lightbox.bind();
+ });
+});
+JS;
+?>
+ <?=$this->inlineScript(\Zend\View\Helper\HeadScript::SCRIPT, $script, 'SET');?>
+<?php endif; ?>
+
+<?php if ($getitConfig['hideNotice']): ?>
+ <?php $script = <<<JS
+$(document).ready(function() {
+ $('.getitbox-notices').addClass('hidden');
+});
+JS;
+?>
+ <?=$this->inlineScript(\Zend\View\Helper\HeadScript::SCRIPT, $script, 'SET');?>
+<?php endif; ?>
+
+<?php if ($openUrlActive): ?>
+ <div class="getitbox-notices">
+ <p class="openUrlPrint"></p>
</div>
+<?php endif; ?>
- <?php if ($openUrlActive): ?>
- <div class="getitbox-notices">
- <div class="openUrlPrint"></div>
- </div>
+<?php if (!empty($getitConfig['notice'])): ?>
+ <div class="getitbox-notices">
+ <?php if ($user && $getitConfig['bossData']): ?>
+ <p class="boss-data-false"></p>
<?php endif; ?>
- <?php if (!empty($getitConfig['notice'])): ?>
- <div class="getitbox-notices">
-
- <?php if ($user && $getitConfig['bossData']): ?>
- <div class="boss-data-false"></div>
- <?php if (!$getitConfig['hideNotice']): ?><br/><?php endif; ?>
- <?php endif; ?>
-
- <?php
- $url = '<a href="'.$this->url('myresearch-account').'?auth_method='.$account->getAuthMethod().'" target="_blank" data-lightbox>'.$this->transEsc('getit_text_3_2').'</a>';
- ?>
- <?php if ($getitConfig['hideNotice']): ?><div class="boss-notice"><?php endif; ?>
- <?=sprintf($getitConfig['notice'], $url)?>
- <?php if ($getitConfig['hideNotice']): ?></div><?php endif; ?>
- </div>
- <?php elseif (!$getitConfig['isAiSidRecord']): ?>
- <div class="getitbox-notices resolver">
- <?php if ($user && $getitConfig['bossData']): ?>
- <div class="boss-data-false"></div>
- <?php if ($getitConfig['hideNotice']): ?><br/><?php endif; ?>
- <?php endif; ?>
- </div>
+ <?php
+ $linkHtml = '<a href="' . $this->url('myresearch-account') . '?auth_method='
+ . $account->getAuthMethod()
+ . '" data-lightbox>' . $this->transEsc('getit_text_3_2') . '</a>';
+ ?>
+ <?php if ($getitConfig['hideNotice']): ?><div class="boss-notice"><?php endif; ?>
+ <?=sprintf($getitConfig['notice'], $linkHtml)?>
+ <?php if ($getitConfig['hideNotice']): ?></div><?php endif; ?>
+ </div>
+<?php elseif (!$getitConfig['isAiSidRecord']): ?>
+ <div class="getitbox-notices resolver">
+ <?php if ($user && $getitConfig['bossData']): ?>
+ <p class="boss-data-false"></p>
<?php endif; ?>
-</table>
-
+ </div>
+<?php endif; ?>
<!-- fid_bbi: recordTab - holdingsils - END -->
diff --git a/themes/fid_bbi/templates/ajax/boss-false.phtml b/themes/fid_bbi/templates/ajax/boss-false.phtml
new file mode 100644
index 00000000000..e8bf4144ea6
--- /dev/null
+++ b/themes/fid_bbi/templates/ajax/boss-false.phtml
@@ -0,0 +1,33 @@
+<!-- fid_bbi: ajax - boss-false -->
+<?php
+/**
+ * This is mostly a copy of themes/boss/templates/ajax/boss-false.phtml (generated)
+ * Changes for FID BBI:
+ * - use external link view helper
+ * - add link icon
+ *
+ * @author Tobias Schäfer <ts@aspectis.net>
+ */
+?>
+<div class="boss-false">
+ <?php if ($this->homeLibrary): ?>
+ <?=$this->translate('boss_default')?>
+ <?php else: ?>
+ <?php if ($this->isISXNZBD): ?>
+ <?php if (!$this->isISIL && !$this->isAddNetwork): ?>
+ <?=$this->translate('boss_false')?>
+ <?php endif; ?>
+ <?php if ($this->isAddNetwork): ?>
+ <?php $linkText = $this->icon('small/external-link') . $this->translate('boss_holding'); ?>
+ <?=$this->externalLink($this->url, $iconHtml . $linkText, ['class' => 'link-with-icon'])?>
+ <br>
+ <a href="<?=$this->url('content-page', ['page' => 'networknotice'])?>" data-lightbox>
+ <?=$this->translate('boss_note')?>
+ </a>
+ <?php endif; ?>
+ <?php else: ?>
+ <?=$this->translate('boss_noISXNZBD')?>
+ <?php endif; ?>
+ <?php endif; ?>
+</div>
+<!-- fid_bbi: ajax - boss-false - END -->
diff --git a/themes/fid_bbi/templates/ajax/boss-true.phtml b/themes/fid_bbi/templates/ajax/boss-true.phtml
new file mode 100644
index 00000000000..5d6830ae3ae
--- /dev/null
+++ b/themes/fid_bbi/templates/ajax/boss-true.phtml
@@ -0,0 +1,29 @@
+<!-- fid_bbi: ajax - boss-true -->
+<?php
+/**
+ * This is mostly a copy of themes/boss/templates/ajax/boss-true.phtml (generated)
+ * Changes for FID BBI:
+ * - use external link view helper
+ * - add link icon
+ *
+ * @author Tobias Schäfer <ts@aspectis.net>
+ */
+?>
+<div class="boss-true">
+ <?php if ($this->homeLibrary): ?>
+ <?=$this->translate('boss_default')?>
+ <?php else: ?>
+ <?php if ($this->isISXNZBD): ?>
+ <?php $linkText = $this->icon('small/external-link') . $this->transEsc('boss_true'); ?>
+ <?=$this->externalLink($this->url, $linkText, ['class' => 'link-with-icon'])?>
+ <?php if (!empty($this->callnumber)): ?>
+ <div class="pda-part-text">
+ <?=$this->transEsc('boss_callnumber')?>: <?=implode(', ', $this->callnumber)?>
+ </div>
+ <?php endif; ?>
+ <?php else: ?>
+ <?=$this->translate('boss_noISXNZBD')?>
+ <?php endif; ?>
+ <?php endif; ?>
+</div>
+<!-- fid_bbi: ajax - boss-true - END -->
diff --git a/themes/fid_bbi/templates/ajax/boss.phtml b/themes/fid_bbi/templates/ajax/boss.phtml
new file mode 100644
index 00000000000..4b1c2cb7b8e
--- /dev/null
+++ b/themes/fid_bbi/templates/ajax/boss.phtml
@@ -0,0 +1,4 @@
+<!-- fid_bbi: ajax - boss -->
+<?php $value = $isISIL ? 'true' : 'false'; ?>
+<?=$this->render("ajax/boss-$value.phtml")?>
+<!-- fid_bbi: ajax - boss - END -->
diff --git a/themes/fid_bbi/templates/ajax/resolverLinks-print.phtml b/themes/fid_bbi/templates/ajax/resolverLinks-print.phtml
new file mode 100644
index 00000000000..77eaccee6aa
--- /dev/null
+++ b/themes/fid_bbi/templates/ajax/resolverLinks-print.phtml
@@ -0,0 +1,95 @@
+<!-- fid-bbi: ajax - resolverlinks-print -->
+<?php
+/**
+ * This is a modified copy of themes/bbi/templates/ajax/resolverLinks-print
+ * Changes for FID BBI:
+ * - change location HTML, remove breaks
+ *
+ * TODO: THIS IS A WORK IN PROGRESS
+ *
+ * @author Tobias Schäfer <ts@aspectis.net>
+ */
+?>
+<?php
+$account = $this->auth()->getManager();
+$user = $account->isLoggedIn();
+?>
+
+<div class="resolverlinks-print">
+ <?php // finc-specific change #8447 - CK ?>
+ <?php $noResolverContent = true; ?>
+ <?php if (!empty($this->messages) && $user): ?>
+ <div class="ezb-message">
+ <?php foreach ($this->messages as $link): ?>
+ <p><?=$this->translate($link)?></p>
+ <?php endforeach; ?>
+ </div>
+ <?php else: ?>
+ <?php
+ $url = '<a href="'.$this->url('myresearch-account').'?auth_method='.$account->getAuthMethod().'" target="_blank" data-lightbox>'.$this->transEsc('getit_text_3_2').'</a>';
+ ?>
+ <?=sprintf($this->translate('getit_text_10'), $url)?>
+ <?php $script = <<<JS
+$(document).ready(function() {
+ // Use to register new lightbox window pda form
+ VuFind.lightbox.bind();
+});
+JS;
+ ?>
+ <?=$this->inlineScript(\Zend\View\Helper\HeadScript::SCRIPT, $script, 'SET');?>
+ <?php endif; ?>
+
+ <?php if (!empty($this->print)): ?>
+ <?php $noResolverContent = false; ?>
+ <div class="openurls">
+ <ul class="unstyled-list">
+ <?php foreach ($this->print as $link): ?>
+ <li>
+ <dl>
+ <?php if (isset($link['Location'])): ?>
+ <dt><?=$this->transEsc('ezb_location')?></dt>
+ <dd><?=$this->escapeHtml($link['Location'])?></dd>
+ <?php endif; ?>
+ <?php if ($link['Signature']): ?>
+ <dt><?=$this->transEsc('Call Number')?></dt>
+ <dd><?=$this->escapeHtml($link['Signature'])?></dd>
+ <?php endif; ?>
+ <?php if (isset($link['Period'])): ?>
+ <dt><?=$this->transEsc('ezb_period')?></dt>
+ <dd><?=$this->escapeHtml($link['Period'])?></dd>
+ <?php endif; ?>
+ <?php if (isset($link['Holding_comment'])): ?>
+ <dt><?=$this->transEsc('ezb_comment')?></dt>
+ <dd><?=$this->escapeHtml($link['Holding_comment'])?></dd>
+ <?php endif; ?>
+ </dl>
+ </li>
+ <?php endforeach; ?>
+ </ul>
+ </div>
+ <?php endif; ?>
+
+ <?php // finc-specific change #9274 - CK ?>
+ <?php if ($noResolverContent): // no content to show, so tell the user ?>
+ <span id="noResolverContentMessage" class="hidden"><?=$this->resolver . ': ' . $this->translate('no_resolver_links')?></span>
+ <script>if ($('#urlsHideable').length) { $('#urlsHideable').show(); } else { $('#noResolverContentMessage').show(); }</script>
+ <?php endif; ?>
+ <?php // finc-specific change #9274 - END ?>
+</div>
+
+<?php // #17831 ?>
+<?php $script = <<<JS
+ $(document).ready(function() {
+ if ($(window).width() >= 1200) {
+ var contentHeight = $('.media').height();
+ var getitboxHeight = $('.getitbox').height();
+ if (getitboxHeight > contentHeight) {
+ $('.media').css('min-height', getitboxHeight+'px');
+ }
+ }
+ });
+JS;
+?>
+<?=$this->inlineScript(\Zend\View\Helper\HeadScript::SCRIPT, $script, 'SET');?>
+<?php // #17831 END ?>
+<!-- fid-bbi: ajax - resolverlinks-print - END -->
diff --git a/themes/fid_bbi/templates/ajax/resolverLinks.phtml b/themes/fid_bbi/templates/ajax/resolverLinks.phtml
index 8e66a0c468d..1d169263bbb 100644
--- a/themes/fid_bbi/templates/ajax/resolverLinks.phtml
+++ b/themes/fid_bbi/templates/ajax/resolverLinks.phtml
@@ -1,79 +1,59 @@
<!-- fid_bbi: ajax - resolverlinks -->
<?php /* this is an exact copy of the finc version to bypass the fid-specific template */?>
-<div>
- <?php /* finc-specific change #8447 - CK */ ?>
+<div class="resolver-links">
+ <?php // finc-specific change #8447 - CK ?>
<?php $noResolverContent = true; ?>
<?php if (!empty($this->electronic)): ?>
<?php $noResolverContent = false; ?>
- <?php /* finc-specific change #8447 - END */ ?>
+ <?php // finc-specific change #8447 - END ?>
<div class="openurls">
- <?php /*
- <strong><?=$this->transEsc('Electronic')?></strong>
- */ ?>
- <ul>
+ <ul class="unstyled-list">
<?php foreach ($this->electronic as $link): ?>
<li>
<?php if (!empty($link['href'])): ?>
- <?php /* finc-specific change #7986 - CK - traffic light */ ?>
- <div <?php if(!empty($link['access'])): ?> data-toggle="tooltip" title="<?=$this->transEsc('resolver_link_access_'.$link['access'])?>" class="traffic-light access-<?=$link['access']?>"<?php endif;?>>
- <span class="first"></span>
- <span class="second"></span>
- <span class="last"></span>
- </div>
- <?php /* finc-specific change #7986 - END */ ?>
- <a href="<?=$this->escapeHtmlAttr($link['href'])?>" title="<?=isset($link['service_type'])?$this->escapeHtmlAttr($link['service_type']):''?>"<?=!empty($link['access'])?' class="access-'.$link['access'].'"':''?>><?=isset($link['title'])?$this->escapeHtml($link['title']):''?></a> <br />
- <?php /* finc-specific change #5334 - CK */ ?>
- <small><?=isset($link['coverage'])?$this->escapeHtml($link['coverage']):''?><?=isset($link['coverageHref'])?' <a href="'.$link['coverageHref'].'" target="_blank">'.$this->translate('Readme').'</a>':''?></small>
- <?php /* finc-specific change #5334 - END */ ?>
- <?php else: ?>
- <?=isset($link['title'])?$this->escapeHtml($link['title']):''?> <?=isset($link['coverage'])?$this->escapeHtml($link['coverage']):''?>
- <?php endif; ?>
- </li>
- <?php endforeach; ?>
- </ul>
- </div>
- <?php endif; ?>
- <?php /* finc-specific change - commented out in #7643 - hide info on print issues - CK */ ?>
- <?php /*
- <?php if (!empty($this->print)): ?>
- <?php $noResolverContent = false; ?>
- <div class="openurls">
- <strong><?=$this->transEsc('Holdings')?></strong>
- <ul>
- <?php foreach ($this->print as $link): ?>
- <li>
- <?php if (!empty($link['href'])): ?>
- <a href="<?=$this->escapeHtmlAttr($link['href'])?>" title="<?=isset($link['service_type'])?$this->escapeHtmlAttr($link['service_type']):''?>"<?=!empty($link['access'])?' class="access-'.$link['access'].'"':''?>><?=isset($link['title'])?$this->escapeHtml($link['title']):''?></a> <?=isset($link['coverage'])?$this->escapeHtml($link['coverage']):''?>
+ <?php
+ $linkText = $this->icon('small/external-link')
+ . $this->transEsc($this->escapeHtml($link['title'] ?? 'unknown'));
+ $linkAttr = [
+ 'class' => 'link-with-icon' . (!empty($link['access']) ? ' access-' . $link['access'] : ''),
+ 'title' => $this->escapeHtmlAttr($link['service_type'] ?? ''),
+ ];
+ ?>
+ <?=$this->externalLink($this->escapeHtmlAttr($link['href']), $linkText, $linkAttr)?>
+ <br>
+
+ <?php // finc-specific change #5334 - CK ?>
+ <small>
+ <?=isset($link['coverage']) ? $this->escapeHtml($link['coverage']) : ''?>
+ <?php if (isset($link['coverageHref'])): ?>
+ <?php $labelHtml = $this->icon('small/external-link') . $this->translate('Readme'); ?>
+ <?=$this->externalLink($link['coverageHref'], $labelHtml)?>
+ <?php endif; ?>
+ </small>
+ <?php // finc-specific change #5334 - END ?>
<?php else: ?>
- <?=isset($link['title'])?$this->escapeHtml($link['title']):''?> <?=isset($link['coverage'])?$this->escapeHtml($link['coverage']):''?>
+ <?=isset($link['title']) ? $this->escapeHtml($link['title']) : ''?>
+ <?=isset($link['coverage']) ? $this->escapeHtml($link['coverage']) : ''?>
<?php endif; ?>
</li>
<?php endforeach; ?>
</ul>
</div>
<?php endif; ?>
- <div class="openurls">
- <?php if (!empty($this->moreOptionsLink)): ?><strong><a href="<?=$this->escapeHtmlAttr($this->moreOptionsLink)?>"><?=$this->transEsc('More options')?></a></strong><?php endif; ?>
- <?php if (!empty($this->services)): ?>
- <ul>
- <?php foreach ($this->services as $link): ?>
- <?php if (!empty($link['href'])): ?>
- <li>
- <a href="<?=$this->escapeHtmlAttr($link['href'])?>" title="<?=isset($link['service_type'])?$this->escapeHtmlAttr($link['service_type']):''?>"<?=!empty($link['access'])?' class="access-'.$link['access'].'"':''?>><?=isset($link['title'])?$this->escapeHtml($link['title']):''?></a>
- </li>
- <?php endif; ?>
- <?php endforeach; ?>
- </ul>
- <?php endif; ?>
- </div>
- */ ?>
- <?php /* finc-specific change - commented out in #7643 - END */ ?>
- <?php /* finc-specific change #9274 - CK */ ?>
+ <?php // finc-specific change #9274 - CK ?>
<?php if ($noResolverContent): // no content to show, so tell the user ?>
- <span id="noResolverContentMessage" class="hidden"><?=$this->resolver . ': ' . $this->translate('no_resolver_links')?></span>
- <script>if ($('#urlsHideable').length) { $('#urlsHideable').show(); } else { $('#noResolverContentMessage').show(); }</script>
+ <span id="noResolverContentMessage" class="hidden">
+ <?=$this->resolver . ': ' . $this->translate('no_resolver_links')?>
+ </span>
+ <script>
+ if ($('#urlsHideable').length) {
+ $('#urlsHideable').show();
+ } else {
+ $('#noResolverContentMessage').show();
+ }
+ </script>
<?php endif; ?>
- <?php /* finc-specific change #9274 - END */ ?>
+ <?php // finc-specific change #9274 - END ?>
</div>
<!-- fid_bbi: ajax - resolverlinks - END -->
diff --git a/themes/fid_bbi/templates/get-it-box/links.phtml b/themes/fid_bbi/templates/get-it-box/links.phtml
index 9e7d12f55e9..a2ebe7a1df1 100644
--- a/themes/fid_bbi/templates/get-it-box/links.phtml
+++ b/themes/fid_bbi/templates/get-it-box/links.phtml
@@ -1,60 +1,50 @@
<!-- fid_bbi: get-it-box - links -->
-<div class="facet-group" id="get-it-links-<?= $getitConfig['accordeonColor'] ?>">
- <button class="title collapsed" id="get-it-button" data-toggle="collapse" href="#side-collapse-links-data">
- <?= $getitConfig['accordeonHeadline'] ?>
- </button>
- <div id="side-collapse-links-data" class="collapse">
- <?php $maxDisplayLength = 50; ?>
+<div class="get-it-box-links" id="get-it-links-<?=$getitConfig['accordeonColor']?>">
+ <div id="side-collapse-links-data">
<?php if (!empty($urls)): ?>
- <?php $isEBCEbooks = $getitConfig['isEBCEBooks']; ?>
- <?php foreach ($urls as $current): ?>
- <?php if ($isEBCEbooks): ?>
- <?php if (preg_match_all('/fid\-medien/', $current['url'])): ?>
- <a href="<?= $this->escapeHtmlAttr($this->proxyUrl($current['url'])) ?>" target="_blank"><i
- class="icon icon-external-link" aria-hidden="true"></i>
- <?= $this->truncate($this->escapeHtml($current['desc']), $maxDisplayLength) ?>
- </a><br/>
- <?php endif; ?>
- <?php else: ?>
- <a href="<?= $this->escapeHtmlAttr($this->proxyUrl($current['url'])) ?>" target="_blank"><i
- class="icon icon-external-link" aria-hidden="true"></i>
- <?= $this->truncate($this->escapeHtml($current['desc']), $maxDisplayLength) ?>
- </a><br/>
- <?php endif; ?>
- <?php endforeach; ?>
+ <ul class="unstyled-list">
+ <?php foreach ($urls as $current): ?>
+ <li>
+ <?php $linkText = $this->icon('small/external-link') . $this->escapeHtml($current['desc']); ?>
+ <?=$this->externalLink($this->escapeHtmlAttr($current['url']), $linkText, ['class' => 'link-with-icon'])?>
+ </li>
+ <?php endforeach; ?>
+ </ul>
<?php endif; ?>
- <?php /* finc-specific snippet - #9274 - replaces if ($openUrlActive): ... - CK */ ?>
+
+ <?php // finc-specific snippet - #9274 - replaces if ($openUrlActive): ... - CK ?>
<?php if ($openUrlActive): ?>
<?php if (empty($urls)): ?>
- <?php $fallbackUrls = $this->driver->tryMethod('getURLs');
- $doiName = $this->driver->tryMethod('getCleanDOI');
- if (!empty($fallbackUrls)): ?>
- <span id="urlsHideable" style="display: none">
- <?php foreach ($fallbackUrls as $current): ?>
- <?php if (preg_match_all('/doi/', $current['url'])): ?>
- <a href="<?=$this->escapeHtmlAttr($this->proxyUrl($current['url']))?>" target="_blank">
- DOI: <?=!empty($doiName) ? $doiName : $current['url'] ?>
- </a>
- <?php endif; ?>
- <?php endforeach; ?>
- </span>
- <?php endif; ?>
+ <?php
+ $fallbackUrls = $this->driver->tryMethod('getURLs');
+ $doiName = $this->driver->tryMethod('getCleanDOI');
+ ?>
+ <?php if (!empty($fallbackUrls)): ?>
+ <p id="urlsHideable" style="display: none">
+ <?php foreach ($fallbackUrls as $current): ?>
+ <?php if (preg_match_all('/doi/', $current['url'])): ?>
+ <a href="<?=$this->escapeHtmlAttr($this->proxyUrl($current['url']))?>">
+ DOI: <?=!empty($doiName) ? $doiName : $current['url'] ?>
+ </a>
+ <?php endif; ?>
+ <?php endforeach; ?>
+ </p>
+ <?php endif; ?>
+ <?php endif; ?>
+
+ <?php
+ // TODO: This leads to duplicate links, so temporarily commented out. We need
+ // to investigate further before actually removing this section, see #19054.
+ // - TS
+ ?>
+ <?php /*
+ <?=$openUrl->renderTemplate()?>
+ */ ?>
+
+ <?php if ($doiActive): ?>
+ <?=$doi->renderTemplate()?>
<?php endif; ?>
- <?= $openUrl->renderTemplate() ?>
- <?php if ($doiActive): ?><?= $doi->renderTemplate() ?><?php endif; ?>
<?php endif; ?>
</div>
</div>
-
-<?php
-$closeAccordeonOnMobile = <<<JS
- $(document).ready(function() {
- if( !$("#search-main-dropdown").is(":visible")) {
- $('#get-it-button').click();
- }
- });
-JS;
-?>
-<?=$this->inlineScript(\Zend\View\Helper\HeadScript::SCRIPT, $closeAccordeonOnMobile, 'SET');?>
-
<!-- fid_bbi: get-it-box - links - END -->
--
GitLab