From e36971c7e192cb0d4e85dcfd8830855e3f4c0f51 Mon Sep 17 00:00:00 2001 From: evykassirer Date: Thu, 5 Dec 2024 15:21:28 -0800 Subject: [PATCH] sidebars: Use pixels for the sidebar spacing. This, as well as increasing the spacing by 5px, ensures that the left sidebar bottom elements aren't overlapped by URLs that the browser shows in the corner when hovering links. --- web/styles/app_variables.css | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/web/styles/app_variables.css b/web/styles/app_variables.css index 114a941b86..8174132613 100644 --- a/web/styles/app_variables.css +++ b/web/styles/app_variables.css @@ -275,8 +275,11 @@ ); --right-sidebar-left-spacing: 5px; /* Used in both left and right sidebar at the bottom, to allow - the user to scroll past the bottom-most elements a little bit. */ - --sidebar-bottom-spacing: 1.4286em; /* 20px / 14px em */ + the user to scroll past the bottom-most elements a little bit. + + Value is chosen to avoid elements overlapping with the browser's URL + display when hovering links, and should not scale with font size. */ + --sidebar-bottom-spacing: 25px; /* We base sidebar row heights on their line heights. Prominent rows include things like headers (e.g., VIEWS)