From 521c9a3243f3aa41f0d1635b15e76a86aade9db5 Mon Sep 17 00:00:00 2001
From: zandercymatics <141044360+zandercymatics@users.noreply.github.com>
Date: Fri, 21 Jun 2024 10:03:50 -0600
Subject: [PATCH] Simplify

---
 src/registrar/assets/sass/_theme/_admin.scss           | 10 ++--------
 .../django/admin/includes/detail_table_fieldset.html   |  4 ++--
 2 files changed, 4 insertions(+), 10 deletions(-)

diff --git a/src/registrar/assets/sass/_theme/_admin.scss b/src/registrar/assets/sass/_theme/_admin.scss
index 6ca57fb6d..b14559050 100644
--- a/src/registrar/assets/sass/_theme/_admin.scss
+++ b/src/registrar/assets/sass/_theme/_admin.scss
@@ -790,9 +790,9 @@ div.dja__model-description{
 .dja-readonly-textarea-container {
     textarea {
         width: 100%;
+        min-width: 610px;
         resize: none;
         cursor: auto;
-        border-width: medium;
 
         &::-webkit-scrollbar {
             background-color: transparent;
@@ -800,16 +800,14 @@ div.dja__model-description{
             width: 12px;
         }
 
+        // Style the scroll bar handle
         &::-webkit-scrollbar-thumb {
             background-color: var(--body-fg);
             border-radius: 99px;
             background-clip: content-box;
             border: 3px solid transparent;
         }
-
     }
-
-    
 }
 
 .max-full {
@@ -821,10 +819,6 @@ div.dja__model-description{
     border-radius: 4px;
 }
 
-.no-border {
-    border: none;
-}
-
 .display-none {
     // Many elements in django admin try to override this, so we need !important.
     display: none !important;
diff --git a/src/registrar/templates/django/admin/includes/detail_table_fieldset.html b/src/registrar/templates/django/admin/includes/detail_table_fieldset.html
index 041a5b3ec..f6313431b 100644
--- a/src/registrar/templates/django/admin/includes/detail_table_fieldset.html
+++ b/src/registrar/templates/django/admin/includes/detail_table_fieldset.html
@@ -69,7 +69,7 @@ This is using a custom implementation fieldset.html (see admin/fieldset.html)
 
 {% block after_help_text %}
     {% if field.field.name == "status" %}
-    <div class="flex-container {% if not filtered_audit_log_entries and original_object.status != "action needed" or not original_object.action_needed_reason and original_object.status == "action needed" and not filtered_audit_log_entries %}display-none{% endif %}" id="dja-status-changelog">
+    <div class="flex-container {% if not filtered_audit_log_entries %}display-none{% endif %}" id="dja-status-changelog">
         <label aria-label="Status changelog"></label>
         <div>
             <div class="usa-table-container--scrollable collapse--dgsimple collapsed" tabindex="0">
@@ -112,7 +112,7 @@ This is using a custom implementation fieldset.html (see admin/fieldset.html)
                     <label class="max-full" for="action_needed_reason_email_view_more">
                         <strong>Auto-generated email (sent to submitter)</strong>
                     </label>
-                    <textarea id="action_needed_reason_email_view_more" cols="40" rows="20" class="no-border {% if not original_object.action_needed_reason %}display-none{% endif %}" readonly>
+                    <textarea id="action_needed_reason_email_view_more" cols="40" rows="20" class="{% if not original_object.action_needed_reason %}display-none{% endif %}" readonly>
                         {{ original_object.action_needed_reason_email }}
                     </textarea>
                     <p id="no-email-message" class="{% if original_object.action_needed_reason %}display-none{% endif %}">No email will be sent</p>