From bae582e2503d598fc8666bbc4ae788ee8dce1e08 Mon Sep 17 00:00:00 2001
From: Ed Madrid <edmadrid@gmail.com>
Date: Tue, 13 Feb 2024 20:36:08 -0500
Subject: [PATCH 1/3] fix layout issues on item pages

---
 site/_includes/components/viewers/osd-iiif.html |  2 +-
 site/_includes/layouts/item.html                | 10 +++++-----
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/site/_includes/components/viewers/osd-iiif.html b/site/_includes/components/viewers/osd-iiif.html
index 9299e31b..673add97 100644
--- a/site/_includes/components/viewers/osd-iiif.html
+++ b/site/_includes/components/viewers/osd-iiif.html
@@ -1,5 +1,5 @@
 {% capture manifest_url %}https://aperitiiif-presentation-api-store.s3.us-east-1.amazonaws.com/bibliopolitica/{{ item.id }}/manifest.json{% endcapture %}
-<div id="map" class="w-100 p-3 z-0 h-full bg-neutral"></div>
+<div id="map" class="w-100 p-3 z-0 h-96 md:h-5/6 bg-neutral"></div>
 <script src="https://cdn.jsdelivr.net/npm/openseadragon@3.1/build/openseadragon/openseadragon.min.js"></script>
 <script type="text/javascript">
   function fullscreenchanged(event) {
diff --git a/site/_includes/layouts/item.html b/site/_includes/layouts/item.html
index c12e54d4..5d625995 100644
--- a/site/_includes/layouts/item.html
+++ b/site/_includes/layouts/item.html
@@ -3,20 +3,20 @@
 ---
 
 <div class="flex p-5 pb-3 items-center">
-  <div class="basis-3/4">
-    <h1 class="text-3xl md:text-4xl font-bold">{{ item.label }}</h1>
+  <div class="basis-5/6">
+    <h1 class="text-2xl md:text-4xl font-bold">{{ item.label }}</h1>
   </div>
-  <div class="basis-1/4 h-12 text-primary text-right flex justify-end">
+  <div class="basis-1/6 h-12 text-primary text-right flex justify-end">
     <a href="{{ pagination.href.previous | default: pagination.href.last | url }}" class="-mr-1 text-neutral-950 hover:text-accent" data-tip="Previous item">{% include "svg/arrow-left.svg" %}</a>
     <a href="{{ pagination.href.next | default: pagination.href.first | url }}" class="text-neutral-950 hover:text-accent" data-tip="Next item">{% include "svg/arrow-right.svg" %}</a>
   </div>
 </div>
 
-<article id="item-content-area" class="px-5 pb-5 md:flex">
+<article id="item-content-area" class="mx-5 mb-5 md:flex border">
   <div class="md:basis-3/5 bg-neutral">
   {% include "components/viewers/osd-iiif.html" %}
   </div>
-  <ul class="p-6 space-y-2 border md:basis-2/5 md:text-lg md:flex md:flex-col md:justify-end">
+  <ul class="p-6 space-y-2 md:border-l md:basis-2/5 md:flex md:flex-col">
     <li><b>Alternate Title(s):</b> {{ item["Alternate title"] | join: '; ' | default: 'N/A' }}</li>
     <li><b>Corporate Creator(s):</b> {{ item["Corporate Creator"] | join: '; ' | default: 'N/A' }}</li>
     <li><b>Person Creator(s):</b> {{ item["Person Creator"] | join: '; ' | default: 'N/A' }}</li>

From 09cbff00ebc2b48d349a60bb7cafeef693358ced Mon Sep 17 00:00:00 2001
From: Ed Madrid <edmadrid@gmail.com>
Date: Tue, 13 Feb 2024 21:15:49 -0500
Subject: [PATCH 2/3] fix issue with inconsistent theme colors

---
 .../components/heros/exhibits-carousel.html       |  2 +-
 .../components/heros/items-carousel.html          |  2 +-
 site/_includes/components/heros/items-grid.html   |  2 +-
 .../components/heros/text-and-featured-items.html |  2 +-
 site/_includes/components/items/carousel.html     |  4 ++--
 site/_includes/layouts/item.html                  |  6 +++---
 site/pages/browse/items.md                        |  2 +-
 tailwind.config.js                                | 15 ++++++++-------
 8 files changed, 18 insertions(+), 17 deletions(-)

diff --git a/site/_includes/components/heros/exhibits-carousel.html b/site/_includes/components/heros/exhibits-carousel.html
index e9ad3399..f9842f1d 100644
--- a/site/_includes/components/heros/exhibits-carousel.html
+++ b/site/_includes/components/heros/exhibits-carousel.html
@@ -1,4 +1,4 @@
-<div class="not-prose bg-neutral mx-[calc(50%-50vw)] my-12 w-auto">
+<div class="not-prose bg-neutral text-neutral-content mx-[calc(50%-50vw)] my-12 w-auto">
   <h2 class="text-center py-6 text-3xl font-bold">Example exhibit carousel title</h2>
   <div class="hero block">
     <div class="hero-content mx-auto bg-neutral">
diff --git a/site/_includes/components/heros/items-carousel.html b/site/_includes/components/heros/items-carousel.html
index 68b12961..7da379b5 100644
--- a/site/_includes/components/heros/items-carousel.html
+++ b/site/_includes/components/heros/items-carousel.html
@@ -1,3 +1,3 @@
-<div class="block not-prose text-base-content bg-neutral mx-[calc(50%-50vw)] my-12 w-auto">
+<div class="block not-prose bg-neutral text-neutral-content mx-[calc(50%-50vw)] my-12 w-auto">
   {% include "components/items/carousel.html" %}
 </div>
\ No newline at end of file
diff --git a/site/_includes/components/heros/items-grid.html b/site/_includes/components/heros/items-grid.html
index d2bf0028..77f5b22f 100644
--- a/site/_includes/components/heros/items-grid.html
+++ b/site/_includes/components/heros/items-grid.html
@@ -1,4 +1,4 @@
-<div class="hero not-prose bg-neutral mx-[calc(50%-50vw)] my-12 w-auto">
+<div class="hero not-prose bg-neutral text-neutral-content mx-[calc(50%-50vw)] my-12 w-auto">
   <div class="hero-content">
     {% include "components/items/grid.html" %}
   </div>
diff --git a/site/_includes/components/heros/text-and-featured-items.html b/site/_includes/components/heros/text-and-featured-items.html
index 7074b7fd..7ba39c4e 100644
--- a/site/_includes/components/heros/text-and-featured-items.html
+++ b/site/_includes/components/heros/text-and-featured-items.html
@@ -1,4 +1,4 @@
-<div class="hero not-prose bg-neutral text-base-content">
+<div class="hero not-prose bg-neutral text-neutral-content">
   <div class="hero-content max-w-full items-start px-6 py-20 flex-col lg:flex-row-reverse">
     <div class="basis-1/2 flex flex-row gap-x-4">
       {% assign featured = items | limit: 2 %}
diff --git a/site/_includes/components/items/carousel.html b/site/_includes/components/items/carousel.html
index 724d3611..2b5df047 100644
--- a/site/_includes/components/items/carousel.html
+++ b/site/_includes/components/items/carousel.html
@@ -7,8 +7,8 @@
       <p>I'm some additional info if needed.</p>
     </div>
     <div class="hidden md:block flex-1 text-right pr-10">
-      <button class="prev h-8 w-8 md:h-12 md:w-12 md:-mr-2 text-neutral-950 hover:text-accent">{% include "svg/arrow-left.svg" %}</button>
-      <button class="next h-8 w-8 md:h-12 md:w-12 text-neutral-950 hover:text-accent">{% include "svg/arrow-right.svg" %}</button>
+      <button class="prev h-8 w-8 md:h-12 md:w-12 md:-mr-2 hover:text-accent">{% include "svg/arrow-left.svg" %}</button>
+      <button class="next h-8 w-8 md:h-12 md:w-12 hover:text-accent">{% include "svg/arrow-right.svg" %}</button>
     </div>
   </div>
 
diff --git a/site/_includes/layouts/item.html b/site/_includes/layouts/item.html
index 5d625995..e3bbb23c 100644
--- a/site/_includes/layouts/item.html
+++ b/site/_includes/layouts/item.html
@@ -6,9 +6,9 @@
   <div class="basis-5/6">
     <h1 class="text-2xl md:text-4xl font-bold">{{ item.label }}</h1>
   </div>
-  <div class="basis-1/6 h-12 text-primary text-right flex justify-end">
-    <a href="{{ pagination.href.previous | default: pagination.href.last | url }}" class="-mr-1 text-neutral-950 hover:text-accent" data-tip="Previous item">{% include "svg/arrow-left.svg" %}</a>
-    <a href="{{ pagination.href.next | default: pagination.href.first | url }}" class="text-neutral-950 hover:text-accent" data-tip="Next item">{% include "svg/arrow-right.svg" %}</a>
+  <div class="basis-1/6 h-12 text-right flex justify-end">
+    <a href="{{ pagination.href.previous | default: pagination.href.last | url }}" class="-mr-1 hover:text-accent" data-tip="Previous item">{% include "svg/arrow-left.svg" %}</a>
+    <a href="{{ pagination.href.next | default: pagination.href.first | url }}" class="hover:text-accent" data-tip="Next item">{% include "svg/arrow-right.svg" %}</a>
   </div>
 </div>
 
diff --git a/site/pages/browse/items.md b/site/pages/browse/items.md
index 9c01db7c..a70d2576 100644
--- a/site/pages/browse/items.md
+++ b/site/pages/browse/items.md
@@ -69,7 +69,7 @@ permalink: "/browse/items/index.html"
               alt=""
               class="max-w-full h-auto"
             >
-            <div class="mt-1 text-base leading-[1.4]">
+            <div class="mt-1 leading-[1.4]">
               {{ item.label }}
             </div>
           </a>
diff --git a/tailwind.config.js b/tailwind.config.js
index 4fb24bb2..65100737 100644
--- a/tailwind.config.js
+++ b/tailwind.config.js
@@ -14,18 +14,19 @@ module.exports = {
       {
         light: {
           ...require("daisyui/src/theming/themes")["light"],
-          primary: "#1a237e",
-          secondary: "#007706",
-          accent: "#1e88e5",
-          neutral: "#f0f0f0"
+          "primary": "#1a237e",
+          "secondary": "#007706",
+          "accent": "#1e88e5",
+          "neutral": "#f0f0f0",
+          "neutral-content": "#333"
         },
       }, 
       {
         dark: {
           ...require("daisyui/src/theming/themes")["dark"],
-          primary: "#1a237e",
-          secondary: "#007706",
-          accent: "#1e88e5"
+          "primary": "#1a237e",
+          "secondary": "#007706",
+          "accent": "#1e88e5"
         },
       },
     ],

From 7f90bfd55d4362aea31dae8214649aff4301e786 Mon Sep 17 00:00:00 2001
From: Ed Madrid <edmadrid@gmail.com>
Date: Tue, 13 Feb 2024 21:18:55 -0500
Subject: [PATCH 3/3] add tooltips to item page nav arrows

---
 site/_includes/layouts/item.html | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/site/_includes/layouts/item.html b/site/_includes/layouts/item.html
index e3bbb23c..6ea9c093 100644
--- a/site/_includes/layouts/item.html
+++ b/site/_includes/layouts/item.html
@@ -7,8 +7,8 @@
     <h1 class="text-2xl md:text-4xl font-bold">{{ item.label }}</h1>
   </div>
   <div class="basis-1/6 h-12 text-right flex justify-end">
-    <a href="{{ pagination.href.previous | default: pagination.href.last | url }}" class="-mr-1 hover:text-accent" data-tip="Previous item">{% include "svg/arrow-left.svg" %}</a>
-    <a href="{{ pagination.href.next | default: pagination.href.first | url }}" class="hover:text-accent" data-tip="Next item">{% include "svg/arrow-right.svg" %}</a>
+    <a href="{{ pagination.href.previous | default: pagination.href.last | url }}" class="-mr-1 hover:text-accent tooltip tooltip-bottom" data-tip="Previous item">{% include "svg/arrow-left.svg" %}</a>
+    <a href="{{ pagination.href.next | default: pagination.href.first | url }}" class="hover:text-accent tooltip tooltip-bottom" data-tip="Next item">{% include "svg/arrow-right.svg" %}</a>
   </div>
 </div>