refs #17950 [master] Change tag <div> with <p> for alerts

* Use tag p instead of div for flashmessages
* Change tag <div> with <p> for alerts
* Review class text-success
** Change <div> with <p>
** Remove margin-bottom
* move newly added files to throwaway theme
* introduce finc specific Flashmessages ViewHelper

co-authored by: "Dorian Merz <merz@ub.uni-leipzig.de>"

module/finc/src/finc/View/Helper/Root/Flashmessages.php

+<?php
+/**
+ * Flash message view helper
+ *
+ * PHP version 7
+ *
+ * Copyright (C) Leipzig University Library 2020.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2,
+ * as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
+ *
+ * @category VuFind
+ * @package  View_Helpers
+ * @author   Heike Reinken <reinken@ub.uni-leipzig.de>
+ * @license  http://opensource.org/licenses/gpl-2.0.php GNU General Public License
+ * @link     https://vufind.org/wiki/development Wiki
+ */
+namespace finc\View\Helper\Root;
+
+/**
+ * Flash message view helper
+ *
+ * @category VuFind
+ * @package  View_Helpers
+ * @author   Heike Reinken <reinken@ub.uni-leipzig.de>
+ * @license  http://opensource.org/licenses/gpl-2.0.php GNU General Public License
+ * @link     https://vufind.org/wiki/development Wiki
+ */
+class Flashmessages extends \VuFind\View\Helper\Bootstrap3\Flashmessages
+{
+    /**
+     * Generate flash message <p>'s with appropriate classes based on message type.
+     * This is basically a copy of the parent method but uses p-Tags instead of divs
+     *
+     * @return string $html
+     */
+    public function __invoke()
+    {
+        $html = '';
+        $namespaces = ['error', 'info', 'success'];
+        foreach ($namespaces as $ns) {
+            $messages = array_merge(
+                $this->fm->getMessages($ns), $this->fm->getCurrentMessages($ns)
+            );
+            foreach (array_unique($messages, SORT_REGULAR) as $msg) {
+                $html .= '<p class="' . $this->getClassForNamespace($ns) . '"';
+                if (isset($msg['dataset'])) {
+                    foreach ($msg['dataset'] as $attr => $value) {
+                        $html .= ' data-' . $attr . '="'
+                            . htmlspecialchars($value) . '"';
+                    }
+                }
+                $html .= '>';
+                // Advanced form:
+                if (is_array($msg)) {
+                    // Use a different translate helper depending on whether
+                    // or not we're in HTML mode.
+                    if (!isset($msg['translate']) || $msg['translate']) {
+                        $helper = (isset($msg['html']) && $msg['html'])
+                            ? 'translate' : 'transEsc';
+                    } else {
+                        $helper = (isset($msg['html']) && $msg['html'])
+                            ? false : 'escapeHtml';
+                    }
+                    $helper = $helper
+                        ? $this->getView()->plugin($helper) : false;
+                    $tokens = $msg['tokens'] ?? [];
+                    $default = $msg['default'] ?? null;
+                    $html .= $helper
+                        ? $helper($msg['msg'], $tokens, $default) : $msg['msg'];
+                } else {
+                    // Basic default string:
+                    $transEsc = $this->getView()->plugin('transEsc');
+                    $html .= $transEsc($msg);
+                }
+                $html .= '</p>';
+            }
+            $this->fm->clearMessages($ns);
+            $this->fm->clearCurrentMessages($ns);
+        }
+        return $html;
+    }
+}

themes/finc-accessibility/templates/Recommend/AlphaBrowseLink.phtml

+<!-- finc-accessibility: Recommend - AlphaBrowseLink -->
+<?php /* #17950 - use <p> for alerts - HR */ ?>
+
+<?php
+  $index = $this->recommend->getIndex();
+  $from = $this->recommend->getQuery();
+  $link = $this->translate(
+    'alphabrowselink_html',
+    [
+      '%%index%%' => $this->transEsc('browse_' . $index),
+      '%%from%%' => $this->escapeHtml($from),
+      '%%url%%' => $this->url('alphabrowse-home')
+        . '?from=' . urlencode($from) . '&amp;source=' . urlencode($index)
+    ]
+  );
+?>
+<p class="alert alert-info"><?=$link?></p>
+<!-- finc-accessibility: Recommend - AlphaBrowseLink - END -->

themes/finc-accessibility/templates/Recommend/Channels.phtml

+<!-- finc-accessibility: Recommend - Channels -->
+<?php /* #17950 - use <p> for alerts - HR */ ?>
+
+<?php
+    $results = $this->recommend->getResults();
+    $link = $this->url('channels-search')
+        . $results->getUrlQuery()->getParams(false)
+        . '&source=' . urlencode($results->getParams()->getSearchClassId());
+?>
+<p class="channels-alert alert alert-info">
+  <a href="<?=$this->escapeHtmlAttr($link)?>"><?=$this->transEsc('channel_expand')?></a>
+</p>
+<!-- finc-accessibility: Recommend - Channels - END -->

themes/finc-accessibility/templates/Recommend/DOI.phtml

+<!-- finc-accessibility: Recommend - DOI -->
+<?php /* #17950 - use <p> for alerts - HR */ ?>
+
+<?php $doi = $this->recommend->getDOI(); if (!empty($doi)): ?>
+  <?php $url = $this->recommend->getURL(); ?>
+  <p class="alert alert-info">
+    <?=$this->translate('doi_detected_html', ['%%url%%' => $url, '%%doi%%' => $doi])?>
+  </p>
+  <?php if ($this->recommend->isFullMatch()): ?>
+    <?php $redirect = 'document.location.href = "' . $this->escapeJs($url) . '";'; ?>
+    <?=$this->inlineScript(\Zend\View\Helper\HeadScript::SCRIPT, $redirect, 'SET')?>
+  <?php endif; ?>
+<?php endif; ?>
+<!-- finc-accessibility: Recommend - DOI - END -->

themes/finc-accessibility/templates/Recommend/RemoveFilters.phtml

+<!-- finc-accessibility: Recommend - RemoveFilters -->
+<?php /* #17950 - use <p> for alerts - HR */ ?>
+
+<?php if ($this->recommend->hasFilters()): ?>
+  <p class="alert alert-info">
+    <?=$this->transEsc('nohit_active_filters')?> 
+    <a href="<?=$this->recommend->getFilterlessUrl()?>"><?=$this->transEsc('nohit_query_without_filters')?></a>
+  </p>
+<?php endif; ?>
+<!-- finc-accessibility: Recommend - RemoveFilters - END -->

themes/finc-accessibility/templates/Recommend/SwitchType.phtml

+<!-- finc-accessibility: Recommend - SwitchType -->
+<?php /* #17950 - use <p> for alerts - HR */ ?>
+
+<?php if ($handler = $this->recommend->getNewHandler()): ?>
+  <p class="alert alert-info">
+    <?=$this->transEsc('widen_prefix')?>
+    <a href="<?=$this->recommend->getResults()->getUrlQuery()->setHandler($handler)?>"><?=$this->transEsc($this->recommend->getNewHandlerName())?></a>.
+  </p>
+<?php endif; ?>
+<!-- finc-accessibility: Recommend - SwitchType - END -->

themes/finc-accessibility/templates/RecordDriver/EDS/result-list.phtml

+<!-- finc-accessibility: RecordDriver - EDS - result-list -->
+<?php /* #17950 - use <p> for alerts - HR */ ?>
+
+<?php
+  $this->headLink()->appendStylesheet('EDS.css');
+  $accessLevel = $this->driver->getAccessLevel();
+  $restrictedView = empty($accessLevel) ? false : true;
+  $coverDetails = $this->record($this->driver)->getCoverDetails('result-list', 'medium', $this->recordLink()->getUrl($this->driver));
+?>
+<?php
+  $thumbnail = false;
+  $thumbnailAlignment = $this->record($this->driver)->getThumbnailAlignment('result');
+  ob_start(); ?>
+  <div class="media-<?=$thumbnailAlignment ?> <?=$this->escapeHtml($coverDetails['size'])?>">
+    <?php if ($coverDetails['cover']): ?>
+      <a href="<?=$this->recordLink()->getUrl($this->driver)?>" class="_record_link">
+        <img src="<?=$this->escapeHtmlAttr($coverDetails['cover'])?>" class="recordcover" alt="<?=$this->transEsc('Cover Image')?>"/>
+      </a>
+    <?php else: ?>
+      <span class="recordcover pt-icon pt-<?=$this->driver->getPubTypeId()?>"></span>
+      <div><?=$this->driver->getPubType()?></div>
+    <?php endif; ?>
+  </div>
+<?php $thumbnail = ob_get_contents(); ?>
+<?php ob_end_clean(); ?>
+<input type="hidden" value="<?=$this->escapeHtmlAttr($this->driver->getUniqueID())?>" class="hiddenId" />
+<input type="hidden" value="<?=$this->escapeHtmlAttr($this->driver->getSourceIdentifier())?>" class="hiddenSource" />
+<div class="media<?=$this->driver->supportsAjaxStatus()?' ajaxItem':''?>">
+  <?php if ($thumbnail && $thumbnailAlignment == 'left'): ?>
+    <?=$thumbnail ?>
+  <?php endif; ?>
+  <div class="media-body">
+    <div class="result-body">
+      <?php $items = $this->driver->getItems();
+        if (isset($items) && !empty($items)):
+          foreach ($items as $item):
+            if (!empty($item)): ?>
+              <div class="resultItemLine1">
+                <?php if('Ti' == $item['Group']): ?>
+                  <a href="<?=$this->recordLink()->getUrl($this->driver)?>" class="title getFull _record_link"  data-view="<?=$this->params->getOptions()->getListViewOption()?>">
+                  <?=$item['Data']?> </a>
+                <?php else:?>
+                  <p>
+                    <b><?=$this->transEsc($item['Label'])?>:</b>
+                    <?=$this->driver->linkUrls($item['Data'])?>
+                  </p>
+                <?php endif;?>
+              </div>
+            <?php endif;
+          endforeach;
+        elseif ($restrictedView): ?>
+          <div class="resultItemLine1">
+            <p>
+              <?=$this->transEsc('This result is not displayed to guests')?>
+              <br />
+              <a class="login" href="<?=$this->url('myresearch-home')?>">
+                <strong><?=$this->transEsc('Login for full access')?></strong>
+              </a>
+            </p>
+          </div>
+        <?php endif; ?>
+
+      <div class="resultItemLine4 custom-links">
+        <?php $customLinks = array_merge($this->driver->getFTCustomLinks(), $this->driver->getCustomLinks());
+        if (!empty($customLinks)): ?>
+          <?php foreach ($customLinks as $customLink): ?>
+            <?php
+              $url = $customLink['Url'] ?? '';
+              $mot = $customLink['MouseOverText'] ?? '';
+              $icon = $customLink['Icon'] ?? '';
+              $name = $customLink['Text'] ?? '';
+            ?>
+            <span>
+              <a href="<?=$this->escapeHtmlAttr($url)?>" target="_blank" title="<?=$this->escapeHtmlAttr($mot)?>" class="custom-link">
+                <?php if ($icon): ?><img src="<?=$this->escapeHtmlAttr($icon)?>" /> <?php endif; ?><?=$this->escapeHtml($name)?>
+              </a>
+            </span>
+          <?php endforeach; ?>
+        <?php endif; ?>
+      </div>
+
+      <?php if ($this->driver->hasHTMLFullTextAvailable()): ?>
+        <a href="<?= $this->recordLink()->getUrl($this->driver, 'fulltext') ?>#html" class="icon html fulltext _record_link" target="_blank">
+          <?=$this->transEsc('HTML Full Text')?>
+        </a>
+        &nbsp; &nbsp;
+      <?php endif; ?>
+
+      <?php if ($this->driver->hasPdfAvailable()): ?>
+        <a href="<?= $this->recordLink()->getUrl($this->driver) . '/PDF'; ?>" class="icon pdf fulltext" target="_blank">
+          <?=$this->transEsc('PDF Full Text')?>
+        </a>
+      <?php endif; ?>
+    </div>
+    <div class="result-links hidden-print">
+      <?php /* Display qrcode if appropriate: */ ?>
+      <?php if ($QRCode = $this->record($this->driver)->getQRCode("results")): ?>
+        <?php
+          // Add JS Variables for QrCode
+          $this->jsTranslations()->addStrings(['qrcode_hide' => 'qrcode_hide', 'qrcode_show' => 'qrcode_show']);
+        ?>
+        <span class="hidden-xs">
+          <i class="fa fa-fw fa-qrcode" aria-hidden="true"></i> <a href="<?=$this->escapeHtmlAttr($QRCode);?>" class="qrcodeLink"><?=$this->transEsc('qrcode_show')?></a>
+          <div class="qrcode hidden">
+            <script type="text/template" class="qrCodeImgTag">
+              <img alt="<?=$this->transEsc('QR Code')?>" src="<?=$this->escapeHtmlAttr($QRCode);?>"/>
+            </script>
+          </div><br/>
+        </span>
+      <?php endif; ?>
+
+      <?php if ($this->cart()->isActiveInSearch() && $this->params->getOptions()->supportsCart() && $this->cart()->isActive()): ?>
+        <?=$this->render('record/cart-buttons.phtml', ['id' => $this->driver->getUniqueId(), 'source' => $this->driver->getSourceIdentifier()]); ?><br/>
+      <?php endif; ?>
+
+      <?php if ($this->userlist()->getMode() !== 'disabled'): ?>
+        <?php /* Add to favorites */ ?>
+        <i class="fa fa-fw fa-star" aria-hidden="true"></i> <a href="<?=$this->recordLink()->getActionUrl($this->driver, 'Save')?>" class="save-record" data-lightbox id="<?=$this->driver->getUniqueId() ?>" title="<?=$this->transEsc('Add to favorites')?>"><?=$this->transEsc('Add to favorites')?></a><br/>
+
+        <?php /* Saved lists */ ?>
+        <p class="savedLists alert alert-info hidden">
+          <strong><?=$this->transEsc("Saved in")?>:</strong>
+        </p>
+      <?php endif; ?>
+
+      <?php /* Hierarchy tree link */ ?>
+      <?php $trees = $this->driver->tryMethod('getHierarchyTrees'); if (!empty($trees)): ?>
+        <?php foreach ($trees as $hierarchyID => $hierarchyTitle): ?>
+          <div class="hierarchyTreeLink">
+            <input type="hidden" value="<?=$this->escapeHtmlAttr($hierarchyID)?>" class="hiddenHierarchyId" />
+            <i class="fa fa-fw fa-sitemap" aria-hidden="true"></i>
+            <a class="hierarchyTreeLinkText" data-lightbox href="<?=$this->recordLink()->getTabUrl($this->driver, 'HierarchyTree')?>?hierarchy=<?=urlencode($hierarchyID)?>#tabnav" title="<?=$this->transEsc('hierarchy_tree')?>">
+              <?=$this->transEsc('hierarchy_view_context')?><?php if (count($trees) > 1): ?>: <?=$this->escapeHtml($hierarchyTitle)?><?php endif; ?>
+            </a>
+          </div>
+        <?php endforeach; ?>
+      <?php endif; ?>
+    </div>
+  </div>
+  <?php if ($thumbnail && $thumbnailAlignment == 'right'): ?>
+    <?=$thumbnail ?>
+  <?php endif; ?>
+</div>
+<!-- finc-accessibility: RecordDriver - EDS - result-list - END -->

themes/finc-accessibility/templates/Related/Channels.phtml

+<!-- finc-accessibility: Related - Channels -->
+<?php /* #17950 - use <p> for alerts - HR */ ?>
+
+<?php
+    $driver = $this->related->getDriver();
+    $link = $this->url('channels-record')
+        . '?id=' . urlencode($driver->getUniqueId())
+        . '&source=' . urlencode($driver->getSourceIdentifier());
+?>
+<p class="channels-alert alert alert-info">
+  <a href="<?=$this->escapeHtmlAttr($link)?>"><?=$this->transEsc('channel_expand')?></a>
+</p>
+<!-- finc-accessibility: Related - Channels - END -->

themes/finc-accessibility/templates/myresearch/delete.phtml

+<!-- finc-accessibility: myresearch - delete -->
+<?php /* #17950 - use <p> for alerts - HR */ ?>
+
+<h2><?=$this->transEsc('delete_selected_favorites')?></h2>
+ <form action="<?=$this->url('myresearch-delete')?>" method="post" name="bulkDelete" data-lightbox-onclose="VuFind.refreshPage">
+  <div id="popupMessages"><?=$this->flashmessages()?></div>
+  <div id="popupDetails">
+    <?php if (!$this->list): ?>
+      <p class="alert alert-info"><?=$this->transEsc("fav_delete_warn") ?></p>
+    <?php else: ?>
+      <h3><?=$this->transEsc("List") ?>: <?=$this->escapeHtml($this->list->title) ?></h3>
+    <?php endif; ?>
+
+    <?php foreach ($this->records as $favorite): ?>
+      <strong><?=$this->transEsc('Title') ?>:</strong>
+      <?=$this->escapeHtml($favorite->getBreadcrumb())?><br />
+    <?php endforeach; ?>
+    <br />
+    <input class="btn btn-primary" type="submit" name="submit" value="<?=$this->transEsc('Delete')?>"/>
+    <?php foreach ($this->deleteIDS as $deleteID): ?>
+      <input type="hidden" name="ids[]" value="<?=$this->escapeHtmlAttr($deleteID)?>" />
+    <?php endforeach; ?>
+      <input type="hidden" name="listID" value="<?=$this->list?$this->escapeHtmlAttr($this->list->id):''?>" />
+  </div>
+</form>
+<!-- finc-accessibility: myresearch - delete - END -->

themes/finc-accessibility/templates/myresearch/login.phtml

+<!-- finc-accessibility: myresearch - login -->
+<?php /* #17950 - use <p> for alerts - HR */ ?>
+
+<?php
+  // Set up page title:
+  $this->headTitle($this->translate('Login'));
+
+  // Set up breadcrumbs:
+  $this->layout()->breadcrumbs = '<li><a href="' . $this->url('myresearch-home') . '">' . $this->transEsc('Your Account') . '</a></li> <li class="active">' . $this->transEsc('Login') . '</li>';
+
+  // Convenience variables:
+  $account = $this->auth()->getManager();
+  $hideLogin = !(is_object($account) && $account->loginEnabled());
+  $offlineMode = $this->ils()->getOfflineMode();
+?>
+
+<?php if ($offlineMode == "ils-offline"): ?>
+  <?=$this->render('Helpers/ils-offline.phtml', ['offlineModeMsg' => 'ils_offline_login_message'])?>
+<?php endif; ?>
+
+<h2 class="lightbox-header"><?=$this->transEsc('Login')?></h2>
+<?=$this->flashmessages()?>
+
+<?php if ($hideLogin): ?>
+  <p class="alert alert-danger"><?=$this->transEsc('login_disabled')?></p>
+<?php else: ?>
+  <?=$this->auth()->getLogin()?>
+<?php endif; ?>
+<!-- finc-accessibility: myresearch - login - END -->

themes/finc-accessibility/templates/record/comments-list.phtml

+<!-- finc-accessibility: record - comments-list -->
+<?php $comments = $this->driver->getComments(); ?>
+<?php if (empty($comments) || count($comments) == 0): ?>
+  <p class="alert alert-info"><?=$this->transEsc('Be the first to leave a comment')?>!</p>
+<?php else: ?>
+  <?php foreach ($comments as $comment): ?>
+    <div class="comment">
+      <div class="comment-name">
+        <strong><?=null === $comment->user_id ? $this->transEsc('comment_anonymous_user') : $this->escapeHtml(trim($comment->firstname . ' ' . $comment->lastname))?></strong><br/>
+        <small>
+          <?=$this->escapeHtml($comment->created)?>
+          <?php if (($user = $this->auth()->isLoggedIn()) && $comment->user_id == $user->id): ?>
+            <a href="<?=$this->recordLink()->getActionUrl($this->driver, 'DeleteComment')?>?delete=<?=urlencode($comment->id)?>" id="recordComment<?=$this->escapeHtml($comment->id)?>" class="delete"><?=$this->transEsc('Delete')?></a>
+          <?php endif; ?>
+        </small>
+      </div>
+      <div class="comment-text">
+        <?=$this->escapeHtml($comment->comment)?>
+      </div>
+    </div>
+  <?php endforeach; ?>
+<?php endif; ?>
+<!-- finc-accessibility: record - comments-list - END -->

themes/finc/js/lightbox.js

@@ -42,7 +42,7 @@ VuFind.register('lightbox', function Lightbox() {
   // Public: Present an alert
   function showAlert(message, _type) {
     var type = _type || 'info';
-    _html('<div class="flash-message alert alert-' + type + '">' + message + '</div>'
+    _html('<p class="flash-message alert alert-' + type + '">' + message + '</p>'
         + '<button class="btn btn-default" data-dismiss="modal">' + VuFind.translate('close') + '</button>');
     _modal.modal('show');
   }
@@ -50,7 +50,7 @@ VuFind.register('lightbox', function Lightbox() {
     var type = _type || 'info';
     _modalBody.find('.flash-message,.fa.fa-spinner').remove();
     _modalBody.find('h2:first-of-type')
-      .after('<div class="flash-message alert alert-' + type + '">' + message + '</div>');
+      .after('<p class="flash-message alert alert-' + type + '">' + message + '</p>');
   }
   function close() {
     _modal.modal('hide');

themes/finc/scss/compiled.scss

@@ -354,6 +354,7 @@ legend {
 
 .text-success {
   background-color: $brand-success;
+  margin-bottom: 0;
 }
 
 .text-danger {

themes/finc/templates/Helpers/branchinfo.phtml

@@ -67,7 +67,7 @@
       </div>
       <div class="row">
           <?php if (isset($info['note'])): ?>
-            <div class="branch-note alert alert-info">
+            <p class="branch-note alert alert-info">
                 <?php if (is_array($info['note'])): ?>
                     <?php foreach ($info['note'] as $notes): ?>
                         <?= $notes ?>
@@ -75,7 +75,7 @@
                 <?php else: ?>
                     <?= $info['note'] ?>
                 <?php endif; ?>
-            </div>
+            </p>
           <?php endif; ?>
       </div>
     </td>

themes/finc/templates/RecordDriver/DefaultRecord/collection-info.phtml

@@ -34,9 +34,9 @@
     <?php endif; ?>
 
     <?php /* Display the lists that this record is saved to */ ?>
-    <div class="savedLists hidden alert alert-info" id="savedLists">
+    <p class="savedLists hidden alert alert-info" id="savedLists">
       <strong><?=$this->transEsc("Saved in")?>:</strong>
-    </div>
+    </p>
 
     <?php /* finc-specific snippet - Begin - CK */
     /* Do not set to 'hidden', make table below collapse in; adapt collection_record.js to keep accordion open - CK */ ?>

themes/finc/templates/RecordDriver/DefaultRecord/core.phtml

@@ -53,9 +53,9 @@
 
     <?php if ($this->userlist()->getMode() !== 'disabled'): ?>
       <?php /* Display the lists that this record is saved to */ ?>
-      <div class="savedLists hidden alert alert-info">
+      <p class="savedLists hidden alert alert-info">
         <strong><?= $this->transEsc("Saved in") ?>:</strong>
-      </div>
+      </p>
     <?php endif; ?>
 
     <?php /* Display Main Details */ ?>

themes/finc/templates/RecordDriver/DefaultRecord/list-entry.phtml

@@ -39,12 +39,12 @@ if ($cover):
           <?php $missing = $this->driver instanceof \VuFind\RecordDriver\Missing; ?>
             <?php if ($missing && $this->driver->isCachedRecord()): ?>
                 <span class="title"><?=$this->record($this->driver)->getTitleHtml()?></span>
-                <div class="alert alert-info">
+                <p class="alert alert-info">
                     <?= $this->translate('record_from_cache')?>
                     <?php if ($queryParams = $this->record($this->driver)->getAdvancedSearchQueryParams()): ?>
                         <br/><a href="<?=$this->url('search-results', [], ['query'=>$queryParams])?>"><?=$this->transEsc('search_cached_record', ['%%title_full%%' => $this->driver->getTitle()])?></a>
                     <?php endif; ?>
-                </div>
+                </p>
             <?php elseif (!$missing): ?>
                 <a href="<?=$this->recordLink()->getUrl($this->driver)?>" class="getFull" data-view="<?=$this->params->getOptions()->getListViewOption() ?>">
                     <span class="title"><?=$this->record($this->driver)->getTitleHtml()?></span>
@@ -260,4 +260,4 @@ if ($cover):
     <?php endif; ?>
   </div>
 </div>
-<!-- finc: RecordDriver - DefaultRecord - list-entry - END -->
+<!-- finc: RecordDriver - DefaultRecord - list-entry - END -->
\ No newline at end of file

themes/finc/templates/RecordDriver/DefaultRecord/result-list.phtml

@@ -215,9 +215,9 @@ $i = 0;
           <?=$block?>
         <?php endif; ?>
         <?php /* Saved lists */ ?>
-        <div class="savedLists alert alert-info hidden">
+        <p class="savedLists alert alert-info hidden">
           <strong><?=$this->transEsc("Saved in")?>:</strong>
-        </div>
+        </p>
       <?php endif; ?>
 
       <?php /* Hierarchy tree link; finc: keep Icon inside link - CK */ ?>

themes/finc/templates/RecordDriver/FincMissing/result-list.phtml

@@ -112,9 +112,9 @@ if ($cover):
           <?=$block?>
         <?php endif; ?>
         <?php /* Saved lists */ ?>
-        <div class="savedLists alert alert-info hidden">
+        <p class="savedLists alert alert-info hidden">
           <strong><?=$this->transEsc("Saved in")?>:</strong>
-        </div>
+        </p>
       <?php endif; ?>
 
       <?=$this->driver->supportsCoinsOpenUrl() ? '<span class="Z3988" title="' . $this->escapeHtmlAttr($this->driver->getCoinsOpenUrl()) . '"></span>' : ''?>

themes/finc/templates/RecordDriver/SolrAI/core.phtml

@@ -53,9 +53,9 @@
 
       <?php if ($this->userlist()->getMode() !== 'disabled'): ?>
         <?php /* Display the lists that this record is saved to */ ?>
-      <div class="savedLists hidden alert alert-info">
+      <p class="savedLists hidden alert alert-info">
           <strong><?= $this->transEsc("Saved in") ?>:</strong>
-      </div>
+      </p>
       <?php endif; ?>
 
       <?php /* Display Main Details */ ?>