Browse Source

feat(insights): Increase main site sidebar width slightly (#71674)

We've added a bunch of new sidebar links, including some longer ones
like "LLM Monitoring". Unfortunately, that link name is too long and
gets truncated if there's a "new" badge present. After much
back-and-forth with Design, the decision was made to _widen the sidebar
temporarily_ during the launch of LLM Monitoring to accommodate the long
string.

We'll put the sidebar back after ~30 days when it's not "new" anymore.
George Gritsouk 9 months ago
parent
commit
dc8019f4e9
2 changed files with 3 additions and 3 deletions
  1. 2 2
      static/app/utils/theme.tsx
  2. 1 1
      static/less/variables.less

+ 2 - 2
static/app/utils/theme.tsx

@@ -776,7 +776,7 @@ const commonTheme = {
 
     containerWidth: '1440px',
     headerHeight: '61px',
-    sidebarWidth: '220px',
+    sidebarWidth: '236px',
   },
 
   sidebar: {
@@ -786,7 +786,7 @@ const commonTheme = {
     badgeSize: '22px',
     smallBadgeSize: '11px',
     collapsedWidth: '70px',
-    expandedWidth: '220px',
+    expandedWidth: '236px',
     mobileHeight: '54px',
     menuSpacing: '15px',
   },

+ 1 - 1
static/less/variables.less

@@ -35,7 +35,7 @@
 // Sets up sidebar offsets
 
 @sidebar-collapsed-width: 70px;
-@sidebar-expanded-width: 220px;
+@sidebar-expanded-width: 236px;
 @sidebar-panel-width: 320px;
 @sidebar-mobile-height: 54px;