From 7b136ae4e3915c033362b2bb9a3c4bfaf1e09caf Mon Sep 17 00:00:00 2001
From: Mathieu Pillard <diox@users.noreply.github.com>
Date: Thu, 9 Nov 2023 21:31:51 +0100
Subject: [PATCH] Fix stats menu items indentation (#21419)

---
 .../templates/stats/addon_report_menu.html    | 62 +++++++++----------
 static/css/zamboni/stats.less                 |  4 ++
 2 files changed, 35 insertions(+), 31 deletions(-)

diff --git a/src/olympia/stats/templates/stats/addon_report_menu.html b/src/olympia/stats/templates/stats/addon_report_menu.html
index eb72825efd..8bac018248 100644
--- a/src/olympia/stats/templates/stats/addon_report_menu.html
+++ b/src/olympia/stats/templates/stats/addon_report_menu.html
@@ -6,41 +6,41 @@
     {% if has_listed_versions %}
     <li data-report="downloads">
       <a href="{{ url('stats.downloads', slug_or_id) }}">{{ _('Downloads') }}</a>
+      <ul>
+        <li data-report="sources">
+          <a href="{{ url('stats.sources', slug_or_id) }}">{{ _('by Source') }}</a>
+        </li>
+        <li data-report="mediums">
+          <a href="{{ url('stats.mediums', slug_or_id) }}">{{ _('by Medium') }}</a>
+        </li>
+        <li data-report="contents">
+          <a href="{{ url('stats.contents', slug_or_id) }}">{{ _('by Content') }}</a>
+        </li>
+        <li data-report="campaigns">
+          <a href="{{ url('stats.campaigns', slug_or_id) }}">{{ _('by Campaign') }}</a>
+        </li>
+      </ul>
     </li>
-    <ul>
-      <li data-report="sources">
-        <a href="{{ url('stats.sources', slug_or_id) }}">{{ _('by Source') }}</a>
-      </li>
-      <li data-report="mediums">
-        <a href="{{ url('stats.mediums', slug_or_id) }}">{{ _('by Medium') }}</a>
-      </li>
-      <li data-report="contents">
-        <a href="{{ url('stats.contents', slug_or_id) }}">{{ _('by Content') }}</a>
-      </li>
-      <li data-report="campaigns">
-        <a href="{{ url('stats.campaigns', slug_or_id) }}">{{ _('by Campaign') }}</a>
-      </li>
-    </ul>
     {% endif %}
     <li data-report="usage">
       <a href="{{ url('stats.usage', slug_or_id) }}">{{ _('Daily Users') }}</a>
+      <ul>
+        <li data-report="versions">
+          <a href="{{ url('stats.versions', slug_or_id) }}">{{ _('by Add-on Version') }}</a>
+        </li>
+        <li data-report="apps">
+          <a href="{{ url('stats.apps', slug_or_id) }}">{{ _('by Application') }}</a>
+        </li>
+        <li data-report="locales">
+          <a href="{{ url('stats.locales', slug_or_id) }}">{{ _('by Language') }}</a>
+        </li>
+        <li data-report="os">
+          <a href="{{ url('stats.os', slug_or_id) }}">{{ _('by Platform') }}</a>
+        </li>
+        <li data-report="countries">
+          <a href="{{ url('stats.countries', slug_or_id) }}">{{ _('by Country') }}</a>
+        </li>
+      </ul>
     </li>
-    <ul>
-      <li data-report="versions">
-        <a href="{{ url('stats.versions', slug_or_id) }}">{{ _('by Add-on Version') }}</a>
-      </li>
-      <li data-report="apps">
-        <a href="{{ url('stats.apps', slug_or_id) }}">{{ _('by Application') }}</a>
-      </li>
-      <li data-report="locales">
-        <a href="{{ url('stats.locales', slug_or_id) }}">{{ _('by Language') }}</a>
-      </li>
-      <li data-report="os">
-        <a href="{{ url('stats.os', slug_or_id) }}">{{ _('by Platform') }}</a>
-      </li>
-      <li data-report="countries">
-        <a href="{{ url('stats.countries', slug_or_id) }}">{{ _('by Country') }}</a>
-      </li>
-    </ul>
   </ul>
 </nav>
diff --git a/static/css/zamboni/stats.less b/static/css/zamboni/stats.less
index 41010b9cf3..2a369bc9b4 100644
--- a/static/css/zamboni/stats.less
+++ b/static/css/zamboni/stats.less
@@ -532,6 +532,10 @@ table.stats-aggregate tbody span.change.minus {
     }
 }
 
+#side-nav ul ul {
+    margin-left: 1em;
+}
+
 #popup-staging {
     display: none;
 }