From 7dc3de84c25493f499e704d080f967eac28a6a6a Mon Sep 17 00:00:00 2001
From: jorge-vitrubio <jorge@vitrubio.net>
Date: Tue, 29 Jun 2021 08:51:17 +0200
Subject: [PATCH] list of sites from table to lists (ul/li) to be able to use
 accordion and more semantic

---
 view/common/block-layout/list-of-sites.phtml | 75 ++++++++++++--------
 view/common/site-list-entry.phtml            | 66 ++++++++---------
 2 files changed, 79 insertions(+), 62 deletions(-)

diff --git a/view/common/block-layout/list-of-sites.phtml b/view/common/block-layout/list-of-sites.phtml
index e4f6fce..651b3ac 100644
--- a/view/common/block-layout/list-of-sites.phtml
+++ b/view/common/block-layout/list-of-sites.phtml
@@ -2,40 +2,55 @@
 $archiveSiteHelper = $this->getHelperPluginManager()->get('archiveSiteMeta');
 ?>
 
-<div class="list-of-sites">
-    <div class="site-list">
+<section class="collections-list">
 
-      <table>
-        <thead>
-          <tr>
-            <th>Title</th>
-            <th>Artist</th>
-            <th>Collection</th>
-            <th>Date</th>
-          </tr>
-        </thead>
-        <tbody>
+  <header>
+    <h2> Latest in collections</h2>
+  </header>
+  <main class="">
+    <article class="">
+      <ul class="accordion" data-responsive-accordion-tabs="accordion small-accordion" data-multi-expand="true" data-allow-all-closed="true" data-deep-link="true">
+        <li class="accordion-item title" data-accordion-item disabled>
+          <a href="#" class="accordion-title">
+            <ul class="">
+              <li class="title">Title</li>
+              <li class="curator">Artist</li>
+              <li class="collection-name">Collection</li>
+              <li class="date">Date</li>
+            </ul>
+          </a>
+          <div class="accordion-content" data-tab-content>
+            <figure>
+            </figure>
+            <p>
+            </p>
+          </div>
+        </li>
 
         <?php
+        // read each omeka site to publish
         foreach($sites as $site):
-            $site_meta_values = $archiveSiteHelper->getSiteMetaValues($site);
-            echo $this->partial('common/site-list-entry', [
-                            'site' => $site,
-                            'showSummary' => $summaries,
-                            'site_meta_values' => $site_meta_values,
-                            'archive_collections' => $archiveSiteHelper->getCollections(),
-            ]);
+          $site_meta_values = $archiveSiteHelper->getSiteMetaValues($site);
+          // table body build file
+          echo $this->partial('common/site-list-entry', [
+            'site' => $site,
+            'showSummary' => $summaries,
+            'site_meta_values' => $site_meta_values,
+            'archive_collections' => $archiveSiteHelper->getCollections(),
+          ]);
         endforeach;
         ?>
-
-      </tbody>
-  </table>
-
-    </div>
-    <?php
-    if (1 == 0):
-    //if ($pagination):
+      </ul>
+    </article>
+  </main>
+  <aside>
+    <nav>
+      <?php
+      if (1 == 0):
+        //if ($pagination):
         echo $this->pagination();
-    endif;
-    ?>
-</div>
+      endif;
+      ?>
+    </nav>
+  </aside>
+</section>
diff --git a/view/common/site-list-entry.phtml b/view/common/site-list-entry.phtml
index a96dbd7..ed8366c 100644
--- a/view/common/site-list-entry.phtml
+++ b/view/common/site-list-entry.phtml
@@ -1,41 +1,43 @@
 <?php
+// publish as a table the content
 $summary = $showSummary ? $site->summary() : null;
+//
 ?>
 
-  <tr class="_expand_site_item">
-    <td>
-      <?= $site->title() ?>
-    </td>
-    <td>
-      <?php if ($site_meta_values != null): ?>
-        <?= $site_meta_values['currator']; ?><br />
-      <?php endif; ?>
-    </td>
-    <td>
-      <?php if ($site_meta_values != null): ?>
-        <?= $archive_collections[$site_meta_values['collection']]; ?>
-      <?php endif; ?>
-    </td>
-    <td>
-      <?php if ($site_meta_values != null): ?>
-        <?= $site_meta_values['project_date']; ?>
-      <?php endif; ?>
-    </td>
-  </tr>
-  <tr style="display:none">
-    <td>
+<li class="accordion-item" data-accordion-item>
+  <a href="#" class="accordion-title">
+    <ul class="">
+      <li class="title">
+        <?= $site->title() ?>
+      </li>
+      <li class="curator">
+        <?php if ($site_meta_values != null): ?>
+          <?= $site_meta_values['currator']; ?><br />
+        <?php endif; ?>
+      </li>
+      <li class="collection-name">
+        <?php if ($site_meta_values != null): ?>
+          <?= $archive_collections[$site_meta_values['collection']]; ?>
+        <?php endif; ?>
+      </li>
+      <li class="date">
+        <?php if ($site_meta_values != null): ?>
+          <?= $site_meta_values['project_date']; ?>
+        <?php endif; ?>
+      </li>
+    </ul>
+  </a>
+  <div class="accordion-content" data-tab-content>
+    <figure>
       <?php if ($site_meta_values != null): ?>
         <img src="<?= $site_meta_values['thumbnail']; ?>" />
       <?php endif; ?>
-    </td>
-    <td colspan="3">
+    </figure>
+    <p>
       <?php if ($summary): ?>
-      <?php echo nl2br($this->escapeHtml($summary)); ?>
-      <br />
+        <?php echo nl2br($this->escapeHtml($summary)); ?>
       <?php endif; ?>
-      <button class="go_to_site_button"
-              onclick="window.location.href = '<?= $site->siteUrl() ?>'">
-        View
-      </button>
-    </td>
-  </tr>
+      <a class="button small" href="<?= $site->siteUrl() ?>">View</a>
+    </p>
+  </div>
+</li>