From 55835092807e1c1ea68ac75895a182921bee4b34 Mon Sep 17 00:00:00 2001
From: Rajit Sarkar <rsar@loc.gov>
Date: Thu, 31 Oct 2024 08:14:19 -0400
Subject: [PATCH 01/12] CONCD-924 Xs don't seem right

---
 concordia/static/scss/base.scss | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/concordia/static/scss/base.scss b/concordia/static/scss/base.scss
index 440bed2ef..52fff9aa6 100644
--- a/concordia/static/scss/base.scss
+++ b/concordia/static/scss/base.scss
@@ -129,6 +129,8 @@ $alert-padding-x: 1rem;
 @import '../../../node_modules/bootstrap/scss/nav';
 @import '../../../node_modules/bootstrap/scss/navbar';
 @import '../../../node_modules/bootstrap/scss/breadcrumb';
+@import '../../../node_modules/bootstrap/scss/alert';
+@import '../../../node_modules/bootstrap/scss/close';
 @import '../../../node_modules/bootstrap/scss/carousel';
 
 // progress
@@ -236,6 +238,11 @@ header.border-bottom {
     }
 }
 
+.alert a {
+    color: inherit;
+    font-size: 1.5rem;
+}
+
 .font-serif {
     font-family: $font-family-serif;
 }

From 075016ba2951f082a9a8b65408139e8712366591 Mon Sep 17 00:00:00 2001
From: Rajit Sarkar <rsar@loc.gov>
Date: Thu, 31 Oct 2024 08:15:16 -0400
Subject: [PATCH 02/12] CONCD-924 Status/contributor count/call to action
 should align with left edge of transcription box (they are currently a little
 indented)

---
 concordia/templates/transcriptions/asset_detail/editor.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/concordia/templates/transcriptions/asset_detail/editor.html b/concordia/templates/transcriptions/asset_detail/editor.html
index f97927114..321401f32 100644
--- a/concordia/templates/transcriptions/asset_detail/editor.html
+++ b/concordia/templates/transcriptions/asset_detail/editor.html
@@ -3,7 +3,7 @@
         {% csrf_token %}
         <input type="hidden" name="supersedes" value="{{ transcription.pk|default:'' }}" />
         <div class="row justify-content-sm-between align-items-end mx-0">
-            <div class="col">
+            <div class="col ps-0">
                 <div id="transcription-status-message">
                     <div id="transcription-status-display" class="row">
                         <h2 id="display-submitted" {% if transcription_status != 'submitted' %}hidden{% endif %}>

From 904b05e90dde979e1dc62457f50179930edf6918 Mon Sep 17 00:00:00 2001
From: Rajit Sarkar <rsar@loc.gov>
Date: Thu, 31 Oct 2024 08:37:32 -0400
Subject: [PATCH 03/12] CONCD-924 workaround for dialog not closing

---
 .../asset_detail/asset_reservation_failure_modal.html           | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/concordia/templates/transcriptions/asset_detail/asset_reservation_failure_modal.html b/concordia/templates/transcriptions/asset_detail/asset_reservation_failure_modal.html
index 1f19afa10..c4b5705dc 100644
--- a/concordia/templates/transcriptions/asset_detail/asset_reservation_failure_modal.html
+++ b/concordia/templates/transcriptions/asset_detail/asset_reservation_failure_modal.html
@@ -2,7 +2,7 @@
     <div class="modal-content">
         <div class="modal-header">
             <h5 class="modal-title">Someone else is already transcribing this page</h5>
-            <button type="button" class="btn-close" data-bs-dismiss="modal" aria-label="Close">
+            <button type="button" class="btn-close" data-bs-dismiss="modal" aria-label="Close" onclick="$('#asset-reservation-failure-modal').hide();">
 
             </button>
         </div>

From bef6d8472700a815e2d7723f6f596787901a799a Mon Sep 17 00:00:00 2001
From: Rajit Sarkar <rsar@loc.gov>
Date: Thu, 31 Oct 2024 08:38:13 -0400
Subject: [PATCH 04/12] CONCD-924 About this campaign header should be centered
 - it's formatted to the left because there isn't a left arrow

---
 concordia/templates/transcriptions/asset_detail/guide.html | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/concordia/templates/transcriptions/asset_detail/guide.html b/concordia/templates/transcriptions/asset_detail/guide.html
index fbd2eca6e..5d4c013af 100644
--- a/concordia/templates/transcriptions/asset_detail/guide.html
+++ b/concordia/templates/transcriptions/asset_detail/guide.html
@@ -31,8 +31,8 @@ <h3 class="px-2 my-1 py-1">How-To Guide</h3>
                 </ul>
             </div>
             <div class="carousel-item container">
-                <div class="border-bottom justify-content-center mb-3 py-1 row">
-                    <div class="col-6 justify-self-center me-4">
+                <div class="border-bottom justify-content-end mb-3 py-1 d-flex">
+                    <div class="col-7 justify-self-center me-4">
                         <h3>About This Campaign</h3>
                     </div>
                     <div class="col-1 pt-2">

From b77ddaacbc5f56b76c5b035b2a1ada1701065e8f Mon Sep 17 00:00:00 2001
From: Rajit Sarkar <rsar@loc.gov>
Date: Thu, 31 Oct 2024 09:10:35 -0400
Subject: [PATCH 05/12] CONCD-924 Show what I can review: Right edge of text
 should align with right edge of Not started filter box

---
 concordia/templates/fragments/_filter-buttons.html      | 2 +-
 concordia/templates/transcriptions/campaign_detail.html | 4 ++--
 concordia/templates/transcriptions/item_detail.html     | 4 ++--
 concordia/templates/transcriptions/project_detail.html  | 4 ++--
 4 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/concordia/templates/fragments/_filter-buttons.html b/concordia/templates/fragments/_filter-buttons.html
index 0e81f124a..7a234d92b 100644
--- a/concordia/templates/fragments/_filter-buttons.html
+++ b/concordia/templates/fragments/_filter-buttons.html
@@ -1,5 +1,5 @@
 {% if user.is_authenticated %}
-    <div class="col-4 mt-4 ps-5 pe-3">
+    <div>
         <input name="radioButtons" type="radio" id="show-all" {% if do_filter %}onclick="filterAssets(false, '{{ all_url }}?{{ sublevel_qs }}')"{% else %}checked{% endif %}>
         <label>Show all</label>
         <input name="radioButtons" type="radio" id="filter-assets" {% if do_filter %}checked{% else %}onclick="filterAssets(true, '{{ filtered_url }}?{{ sublevel_qs }}')"{% endif %} class="ml-1">
diff --git a/concordia/templates/transcriptions/campaign_detail.html b/concordia/templates/transcriptions/campaign_detail.html
index 167df3e6d..cf0b0830b 100644
--- a/concordia/templates/transcriptions/campaign_detail.html
+++ b/concordia/templates/transcriptions/campaign_detail.html
@@ -118,8 +118,8 @@ <h4 class="mb-3">Helpful Links</h4>
                     {% endif %}
                 </div>
             </div>
-            <div class="row">
-                <div class="col-12 col-lg mt-4">
+            <div class="d-flex justify-content-between mt-4">
+                <div>
                     <h3>Filter pages:</h2>
                 </div>
                 {% url 'transcriptions:campaign-detail' campaign.slug as all_assets %}
diff --git a/concordia/templates/transcriptions/item_detail.html b/concordia/templates/transcriptions/item_detail.html
index 484cc6d5f..69da66dc2 100644
--- a/concordia/templates/transcriptions/item_detail.html
+++ b/concordia/templates/transcriptions/item_detail.html
@@ -38,8 +38,8 @@ <h1>{{ item.title }}</h1>
             </div>
         </div>
         {% include "fragments/transcription-progress-row.html" %}
-        <div class="row">
-            <div class="col-12 col-lg mt-4">
+        <div class="d-flex justify-content-between mt-4">
+            <div>
                 <h3>Filter pages:</h2>
             </div>
             {% url 'transcriptions:item-detail' campaign.slug project.slug item.item_id as all_assets %}
diff --git a/concordia/templates/transcriptions/project_detail.html b/concordia/templates/transcriptions/project_detail.html
index 1b8490c61..ee48d3392 100644
--- a/concordia/templates/transcriptions/project_detail.html
+++ b/concordia/templates/transcriptions/project_detail.html
@@ -35,8 +35,8 @@ <h1>{{ project.title }}</h1>
                 <div class="hero-text">{{ project.description|safe }}</div>
             </div>
         </div>
-        <div class="row">
-            <div class="col-12 col-lg mt-4">
+        <div class="d-flex justify-content-between mt-4">
+            <div>
                 <h3>Filter pages:</h2>
             </div>
             {% url 'transcriptions:project-detail' campaign.slug project.slug as all_assets %}

From f7783adfd9d55638caf5f7f1a4e23215c292e2e4 Mon Sep 17 00:00:00 2001
From: Rajit Sarkar <rsar@loc.gov>
Date: Thu, 31 Oct 2024 09:38:25 -0400
Subject: [PATCH 06/12] CONCD-924 Radio button styling doesn't match 'show what
 i can review' button

---
 concordia/templates/fragments/recent-pages.html | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/concordia/templates/fragments/recent-pages.html b/concordia/templates/fragments/recent-pages.html
index 2f33a69f2..ee3a218f1 100644
--- a/concordia/templates/fragments/recent-pages.html
+++ b/concordia/templates/fragments/recent-pages.html
@@ -18,12 +18,12 @@ <h2>Recent Pages Worked On</h2>
         <button type="submit" class="btn btn-primary rounded-0 p-2">Go</button>
     </div>
     <div class="d-flex flex-row pt-1">
-        <div class="form-check">
-            <input class="form-check-input" type="radio" name="flexRadioDefault" id="flexRadioDefault1" onclick="sortDateDescending();"{% if order_by != 'date-ascending' %} checked{% endif %}>
+        <div>
+            <input type="radio" name="flexRadioDefault" id="flexRadioDefault1" onclick="sortDateDescending();"{% if order_by != 'date-ascending' %} checked{% endif %}>
             <label class="form-check-label" for="flexRadioDefault1">Sort by Newest</label>
         </div>
-        <div class="form-check" style="margin-left: 1.6rem;">
-            <input class="form-check-input" type="radio" name="flexRadioDefault" id="flexRadioDefault2" onclick="sortDateAscending();"{% if order_by == 'date-ascending'%} checked{% endif %}>
+        <div style="margin-left: 1.6rem;">
+            <input type="radio" name="flexRadioDefault" id="flexRadioDefault2" onclick="sortDateAscending();"{% if order_by == 'date-ascending'%} checked{% endif %}>
             <label class="form-check-label" for="flexRadioDefault2">Sort by Oldest</label>
         </div>
     </div>

From 677a74f74e0262d0a99116ae21b771031c8342e3 Mon Sep 17 00:00:00 2001
From: Rajit Sarkar <rsar@loc.gov>
Date: Thu, 31 Oct 2024 10:29:18 -0400
Subject: [PATCH 07/12] CONCD-924 Banner styling is different - link shows as
 box within the box of the banner instead of just a hyperlink within the
 banner

---
 concordia/static/scss/base.scss |  6 +++++-
 concordia/templates/home.html   | 17 ++++++++++-------
 2 files changed, 15 insertions(+), 8 deletions(-)

diff --git a/concordia/static/scss/base.scss b/concordia/static/scss/base.scss
index 52fff9aa6..3b0eee49f 100644
--- a/concordia/static/scss/base.scss
+++ b/concordia/static/scss/base.scss
@@ -236,9 +236,13 @@ header.border-bottom {
             color: $dark;
         }
     }
+
+    &.alert-success {
+        background-color: $green;
+    }
 }
 
-.alert a {
+.alert a[type='button'] {
     color: inherit;
     font-size: 1.5rem;
 }
diff --git a/concordia/templates/home.html b/concordia/templates/home.html
index f99f41813..112682a45 100644
--- a/concordia/templates/home.html
+++ b/concordia/templates/home.html
@@ -15,14 +15,17 @@
     {% if SHOW_BANNER and banner %}
         <div id="homepage-contribute-container" class="container my-4">
             <div class="px-default ms-md-3 mb-5">
-                <div class="alert {{ banner.alert_class }} alert-dismissible w-100 text-center" id="banner-{{ banner.slug }}" role="alert">
-                    <a class="btn {{ banner.btn_class }}" href="{{ banner.link }}"{% if banner.open_in_new_window_tab %} target="_blank"{% endif %}>
-                        {{ banner.text }}
+                <div class="alert {{ banner.alert_class }} alert-dismissible w-100 d-flex justify-content-between" id="banner-{{ banner.slug }}" role="alert">
+                    <div></div>
+                    <div>
+                        <a class="btn {{ banner.btn_class }}" href="{{ banner.link }}"{% if banner.open_in_new_window_tab %} target="_blank"{% endif %}>
+                            {{ banner.text }}
+                        </a>
+                        <button type="button" class="btn btn-dark" id="no-interface-banner">Don't display this again</button>
+                    </div>
+                    <a type="button" data-bs-dismiss="alert" aria-label="Close">
+                        <span class="fas fa-times text-white"></span>
                     </a>
-                    <button type="button" class="btn btn-dark" id="no-interface-banner">Don't display this again</button>
-                    <button type="button" class="btn-close" data-bs-dismiss="alert" aria-label="Close">
-
-                    </button>
                 </div>
             </div>
         </div>

From 20e56adcb1a248d4497ac9221474de83694a547f Mon Sep 17 00:00:00 2001
From: Rajit Sarkar <rsar@loc.gov>
Date: Thu, 31 Oct 2024 15:33:46 -0400
Subject: [PATCH 08/12] CONCD-924 Status and contributor count disappear after
 you click the save button

---
 concordia/static/js/src/contribute.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/concordia/static/js/src/contribute.js b/concordia/static/js/src/contribute.js
index 046b796c9..de599785a 100644
--- a/concordia/static/js/src/contribute.js
+++ b/concordia/static/js/src/contribute.js
@@ -305,8 +305,8 @@ function setupPage() {
                 .attr('hidden', 'hidden')
                 .filter('#message-inprogress')
                 .removeAttr('hidden');
-            messageChildren
-                .filter('#message-contributors')
+            $('#transcription-status-display').removeAttr('hidden');
+            $('#message-contributors')
                 .removeAttr('hidden')
                 .find('#message-contributors-num')
                 .html(responseData.asset.contributors);

From adcb882fb0cd9164ac5206181a27fd3694bd562e Mon Sep 17 00:00:00 2001
From: Rajit Sarkar <rsar@loc.gov>
Date: Thu, 31 Oct 2024 15:40:32 -0400
Subject: [PATCH 09/12] CONCD-924 Carousel header should be bolded

---
 concordia/templates/home.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/concordia/templates/home.html b/concordia/templates/home.html
index 112682a45..96ac8d695 100644
--- a/concordia/templates/home.html
+++ b/concordia/templates/home.html
@@ -77,7 +77,7 @@ <h3 class="mt-3 text-uppercase">Review</h3>
                     <div class="carousel-item {% if forloop.first %} active {% endif %}" {% if slide.overlay_position == "right" %} data-overlay-position="top-right" {% endif %} data-bs-title="{{ slide.headline }}" data-hero-text="{{ slide.body }}" data-link-url="{{ slide.lets_go_url }}">
                         <img class="d-block img-fluid" src="{{ MEDIA_URL }}{{ slide.carousel_image }}" alt="{{ slide.image_alt_text }}" width="1200" height="480">
                         <div class="carousel-overlay text-center d-flex flex-column justify-content-around align-items-center">
-                            <h2 class="h1 title mb-1">{{ slide.headline }}</h2>
+                            <h2 class="h1 title mb-1 fw-bold">{{ slide.headline }}</h2>
                             <p class="hero-text mx-auto">{{ slide.body }}</p>
                             <a class="btn btn-primary px-4" href="{{ slide.lets_go_url }}">LET'S GO!</a>
                         </div>

From a7a82b96364823cfdeb34971e70c127c03f7ca70 Mon Sep 17 00:00:00 2001
From: Rajit Sarkar <rsar@loc.gov>
Date: Thu, 31 Oct 2024 15:56:10 -0400
Subject: [PATCH 10/12] CONCD-924 project image thumbnail size

---
 concordia/static/scss/base.scss | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/concordia/static/scss/base.scss b/concordia/static/scss/base.scss
index 3b0eee49f..809e127e0 100644
--- a/concordia/static/scss/base.scss
+++ b/concordia/static/scss/base.scss
@@ -716,10 +716,13 @@ li.breadcrumb-item.active {
     margin: 0 -6px;
 }
 
+.row .concordia-object-card-container {
+    width: 90%;
+}
+
 .concordia-object-card-container {
     display: flex;
     flex-wrap: wrap;
-    width: 90%;
     padding-top: 0.25em;
 }
 

From 792417e85ed2e1297abcdca69d461a20c27766f3 Mon Sep 17 00:00:00 2001
From: Rajit Sarkar <rsar@loc.gov>
Date: Fri, 1 Nov 2024 08:15:54 -0400
Subject: [PATCH 11/12] CONCD-924 fixed a bug where the guide was sliding twice

---
 concordia/static/scss/base.scss                            | 5 +++++
 concordia/templates/transcriptions/asset_detail/guide.html | 6 +++---
 2 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/concordia/static/scss/base.scss b/concordia/static/scss/base.scss
index 809e127e0..0c3393378 100644
--- a/concordia/static/scss/base.scss
+++ b/concordia/static/scss/base.scss
@@ -1010,6 +1010,11 @@ body .disabled > .page-link {
     width: 14px;
 }
 
+#previous-guide,
+#next-guide {
+    cursor: pointer;
+}
+
 .aspect-ratio-box {
     height: 0;
     overflow: hidden;
diff --git a/concordia/templates/transcriptions/asset_detail/guide.html b/concordia/templates/transcriptions/asset_detail/guide.html
index 5d4c013af..ed0fede3e 100644
--- a/concordia/templates/transcriptions/asset_detail/guide.html
+++ b/concordia/templates/transcriptions/asset_detail/guide.html
@@ -36,7 +36,7 @@ <h3 class="px-2 my-1 py-1">How-To Guide</h3>
                         <h3>About This Campaign</h3>
                     </div>
                     <div class="col-1 pt-2">
-                        <a class="fw-bold ms-3" id="next-guide" href="#guide-carousel" data-bs-slide="next">></a>
+                        <a class="fw-bold ms-3" id="next-guide" data-bs-target="#guide-carousel" data-bs-slide="next">></a>
                     </div>
                 </div>
                 <div class="guide-body">
@@ -67,14 +67,14 @@ <h5 class="pt-3">Helpful Links</h5>
                 <div class="carousel-item container" id="pane-{{ forloop.counter }}">
                     <div class="border-bottom justify-content-center mb-3 py-1 row">
                         <div class="col-1 pt-2 prev-guide">
-                            <a class="fw-bold" id="previous-guide" href="#guide-carousel" data-bs-slide="prev"><</a>
+                            <a class="fw-bold" id="previous-guide" data-bs-target="#guide-carousel" data-bs-slide="prev"><</a>
                         </div>
                         <div class="col text-center ps-2">
                             <h3>{{ guide.title }}</h3>
                         </div>
                         <div class="col-1 pt-2 next-guide">
                             {% if not forloop.last %}
-                                <a class="fw-bold" id="next-guide" href="#guide-carousel" data-bs-slide="next">></a>
+                                <a class="fw-bold" id="next-guide" data-bs-target="#guide-carousel" data-bs-slide="next">></a>
                             {% endif %}
                         </div>
                     </div>

From 8e7743a36301dc480d84bb7c3716e9e87cdbe5f6 Mon Sep 17 00:00:00 2001
From: Rajit Sarkar <rsar@loc.gov>
Date: Fri, 1 Nov 2024 10:15:16 -0400
Subject: [PATCH 12/12] CONCD-924 syntax error in the JS console

---
 concordia/templates/home.html | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/concordia/templates/home.html b/concordia/templates/home.html
index 96ac8d695..ae47b9be5 100644
--- a/concordia/templates/home.html
+++ b/concordia/templates/home.html
@@ -223,8 +223,13 @@ <h2 class="text-center font-serif mb-4">Campaigns: <small>Choose which collectio
             }
         }
         $("#no-interface-banner").click(function(event) {
-            storage.setItem(event.target.parentElement.id, true);
-            $('#' + event.target.parentElement.id).attr("hidden", true);
+            var banner = event.target.parentElement.parentElement;
+            if (banner.hasAttribute('id')) {
+                storage.setItem(banner.id, true);
+                var element = document.getElementById(banner.id);
+                element.classList.remove("d-flex");
+                element.attr("hidden", "hidden");
+            }
         });
     </script>
 {% endblock body_scripts %}