From 13d0b247fe3a38d22cb2246867b4d6a844fc0593 Mon Sep 17 00:00:00 2001
From: zandercymatics <141044360+zandercymatics@users.noreply.github.com>
Date: Mon, 10 Jun 2024 13:55:20 -0600
Subject: [PATCH] PR suggestions

---
 src/registrar/assets/sass/_theme/_admin.scss | 2 +-
 src/registrar/templates/admin/analytics.html | 2 +-
 src/registrar/utility/csv_export.py          | 3 ++-
 3 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/registrar/assets/sass/_theme/_admin.scss b/src/registrar/assets/sass/_theme/_admin.scss
index 042610ad6..c07a4955e 100644
--- a/src/registrar/assets/sass/_theme/_admin.scss
+++ b/src/registrar/assets/sass/_theme/_admin.scss
@@ -762,7 +762,7 @@ div.dja__model-description{
     color: #{$dhs-blue-60};
 }
 
-.analytics-button-group {
+.wrapped-button-group {
     // This button group has too many items
     flex-wrap: wrap;
     // Fix a weird spacing issue with USWDS a buttons in DJA
diff --git a/src/registrar/templates/admin/analytics.html b/src/registrar/templates/admin/analytics.html
index 8a7d93a24..a083dc0b3 100644
--- a/src/registrar/templates/admin/analytics.html
+++ b/src/registrar/templates/admin/analytics.html
@@ -27,7 +27,7 @@
       <div class="module height-full">
         <h2>Current domains</h2>
         <div class="padding-top-2 padding-x-2">
-          <ul class="usa-button-group analytics-button-group">
+          <ul class="usa-button-group wrapped-button-group">
             <li class="usa-button-group__item">
                 <a href="{% url 'export_data_type' %}" class="button text-no-wrap" role="button">
                   <svg class="usa-icon" aria-hidden="true" focusable="false" role="img" width="24" height="24">
diff --git a/src/registrar/utility/csv_export.py b/src/registrar/utility/csv_export.py
index b80601a77..43e1c7edc 100644
--- a/src/registrar/utility/csv_export.py
+++ b/src/registrar/utility/csv_export.py
@@ -817,7 +817,8 @@ class DomainRequestExport:
         # Annotations are custom columns returned to the queryset (AKA: computed in the DB).
         annotations = cls._full_domain_request_annotations()
 
-        # .values can't go two levels deep (just returns the id) - so we have to include this.
+        # The .values returned from annotate_and_retrieve_fields can't go two levels deep 
+        # (just returns the field id of say, "creator") - so we have to include this.
         additional_values = [
             "requested_domain__name",
             "federal_agency__agency",