From 6c3707f48459a40c5aa5122ca049a30348d64f47 Mon Sep 17 00:00:00 2001
From: Shubham Padia <shubham@zulip.com>
Date: Wed, 10 Jul 2024 03:10:04 +0000
Subject: [PATCH] left_sidebar: Move DM section header id to top level in css
 file.

This is a refactor and should not introduce any visual changes.
Fixes https://github.com/zulip/zulip/pull/30707#discussion_r1669110661.
---
 web/styles/left_sidebar.css | 80 ++++++++++++++++++-------------------
 1 file changed, 40 insertions(+), 40 deletions(-)

diff --git a/web/styles/left_sidebar.css b/web/styles/left_sidebar.css
index 0f594d370f..7359208834 100644
--- a/web/styles/left_sidebar.css
+++ b/web/styles/left_sidebar.css
@@ -198,51 +198,51 @@ li.show-more-topics {
     }
 }
 
+#direct-messages-section-header {
+    grid-template-columns: 0 15px minmax(0, 1fr) max-content 30px 0;
+    grid-template-rows: var(--line-height-sidebar-row-prominent);
+    /* TODO: Rewrite the `--left-sidebar-collapse-widget-gutter`
+       value and the styles dependent on it so that this kind of
+       1990s layout shenanigans is made unnecessary. */
+    margin-left: -3px;
+    cursor: pointer;
+    white-space: nowrap;
+
+    #toggle-direct-messages-section-icon {
+        grid-area: starting-anchor-element;
+    }
+
+    .left-sidebar-title {
+        grid-area: row-content;
+    }
+
+    .heading-markers-and-controls {
+        grid-area: markers-and-controls;
+        display: flex;
+        gap: 5px;
+        align-items: center;
+    }
+
+    #show-all-direct-messages {
+        width: var(--left-sidebar-header-icon-width);
+        display: flex;
+        align-items: center;
+        justify-content: center;
+        opacity: 0.7;
+        text-decoration: none;
+        color: inherit;
+
+        &:hover {
+            opacity: 1;
+        }
+    }
+}
+
 .direct-messages-container {
     /* Properly offset all the grid rows
        in the DM section. */
     margin-right: 12px;
 
-    &#direct-messages-section-header {
-        grid-template-columns: 0 15px minmax(0, 1fr) max-content 30px 0;
-        grid-template-rows: var(--line-height-sidebar-row-prominent);
-        /* TODO: Rewrite the `--left-sidebar-collapse-widget-gutter`
-           value and the styles dependent on it so that this kind of
-           1990s layout shenanigans is made unnecessary. */
-        margin-left: -3px;
-        cursor: pointer;
-        white-space: nowrap;
-
-        #toggle-direct-messages-section-icon {
-            grid-area: starting-anchor-element;
-        }
-
-        .left-sidebar-title {
-            grid-area: row-content;
-        }
-
-        .heading-markers-and-controls {
-            grid-area: markers-and-controls;
-            display: flex;
-            gap: 5px;
-            align-items: center;
-        }
-
-        #show-all-direct-messages {
-            width: var(--left-sidebar-header-icon-width);
-            display: flex;
-            align-items: center;
-            justify-content: center;
-            opacity: 0.7;
-            text-decoration: none;
-            color: inherit;
-
-            &:hover {
-                opacity: 1;
-            }
-        }
-    }
-
     & ul.dm-list {
         list-style-type: none;
         font-weight: 400;