From b1f7da4fe3befc571f8bed4007396d4935ed74ae Mon Sep 17 00:00:00 2001 From: Kris Date: Thu, 30 Jun 2022 16:14:17 -0400 Subject: [PATCH] UX: minor experimental sidebar alignment changes (#17292) --- app/assets/stylesheets/common/base/sidebar.scss | 10 +++++++++- app/assets/stylesheets/desktop/discourse.scss | 6 +++++- 2 files changed, 14 insertions(+), 2 deletions(-) diff --git a/app/assets/stylesheets/common/base/sidebar.scss b/app/assets/stylesheets/common/base/sidebar.scss index 9d3b064ed51..1109790e77c 100644 --- a/app/assets/stylesheets/common/base/sidebar.scss +++ b/app/assets/stylesheets/common/base/sidebar.scss @@ -4,11 +4,19 @@ .header-sidebar-toggle { margin-right: 1em; - margin-left: -10px; + + @media screen and (min-width: 1300px) { + // extending the toggle beyond the page when space allows + // for better logo alignment with content + body:not(.has-sidebar-page) & { + margin-left: -3.7em; + } + } button { position: relative; font-size: var(--font-up-2); + padding: 0.5em; .discourse-no-touch & { &:hover { diff --git a/app/assets/stylesheets/desktop/discourse.scss b/app/assets/stylesheets/desktop/discourse.scss index 5eea6cfcfdc..addabb431a2 100644 --- a/app/assets/stylesheets/desktop/discourse.scss +++ b/app/assets/stylesheets/desktop/discourse.scss @@ -209,13 +209,17 @@ body.has-sidebar-page { max-width: calc(var(--d-sidebar-width) + var(--d-max-width)); } .d-header .wrap { - padding-left: 1.75em; + padding-left: 0; } #main-outlet-wrapper { grid-template-columns: var(--d-sidebar-width) minmax(0, 1fr); gap: 0 2em; padding-left: 0; } + .extra-info-wrapper { + // align docked header title with post content + margin-left: 7.7em; + } } body.sidebar-animate {