From 3759c179af3bd48f44f8a244462611e842b3c273 Mon Sep 17 00:00:00 2001
From: alexisben <alex@noesya.coop>
Date: Thu, 28 Dec 2023 10:08:52 +0100
Subject: [PATCH] handle sidebar context

---
 assets/sass/_theme/_utils.sass               | 43 +++++++++-----------
 assets/sass/_theme/design-system/layout.sass |  5 +++
 assets/sass/_theme/sections/papers.sass      | 17 ++++----
 3 files changed, 33 insertions(+), 32 deletions(-)

diff --git a/assets/sass/_theme/_utils.sass b/assets/sass/_theme/_utils.sass
index 82772685..65d4a7f6 100644
--- a/assets/sass/_theme/_utils.sass
+++ b/assets/sass/_theme/_utils.sass
@@ -11,7 +11,8 @@ $space-unit: 4 !default
 
 @mixin in-page-with-sidebar
     @include media-breakpoint-up(desktop)
-        body:not(.full-width) &
+        body:not(.full-width) &,
+        &:is(body):not(.full-width)
             @content
 
 @mixin in-page-without-sidebar
@@ -443,28 +444,24 @@ $space-unit: 4 !default
     src: $src
 
 @mixin sidebar($side: start)
-    @include media-breakpoint-down(desktop)
-        padding: 0 half($grid-gutter-sm)
-        margin-bottom: $spacing3
-    @include media-breakpoint-up(desktop)
-        @if $side == start
-            @include container-margin-left
-            left: 0
-        @else
-            @include container-margin-right
-            right: 0
-        margin-top: 0
-        top: 0
-        height: 100%
-        position: absolute
-        width: col-outside-container(4)
-        & > div
-            @include sticky($spacing1)
-        .toc-container
-            border-top: 1px solid $color-border
-            padding-top: $spacing1
-            position: static
-            margin-left: 0
+    @if $side == start
+        @include container-margin-left
+        left: 0
+    @else
+        @include container-margin-right
+        right: 0
+    margin-top: 0
+    top: 0
+    height: 100%
+    position: absolute
+    width: col-outside-container(4)
+    & > div
+        @include sticky($spacing1)
+    .toc-container
+        border-top: 1px solid $color-border
+        padding-top: $spacing1
+        position: static
+        margin-left: 0
 
 // Old browsers support
 
diff --git a/assets/sass/_theme/design-system/layout.sass b/assets/sass/_theme/design-system/layout.sass
index 722758b5..761441b3 100644
--- a/assets/sass/_theme/design-system/layout.sass
+++ b/assets/sass/_theme/design-system/layout.sass
@@ -67,6 +67,11 @@ ol
         .heading h2, .block .block-content
             padding-left: offset(4)
 
+.section-sidebar
+    @include media-breakpoint-down(desktop)
+        padding: 0 half($grid-gutter-sm)
+        margin-bottom: $spacing3
+
 details
     &:not([open]):hover
        summary::after
diff --git a/assets/sass/_theme/sections/papers.sass b/assets/sass/_theme/sections/papers.sass
index de86d37a..30874106 100644
--- a/assets/sass/_theme/sections/papers.sass
+++ b/assets/sass/_theme/sections/papers.sass
@@ -116,18 +116,17 @@
                         flex-direction: column
                         span
                             order: 3
-    .document-content
-        &.sidebar-on-start
-            .section-sidebar
-                @include sidebar(start)
-            @include media-breakpoint-up(desktop)
+    @include in-page-with-sidebar
+        .document-content
+            &.sidebar-on-start
+                .section-sidebar
+                    @include sidebar(start)
                 .container
                     .content
                         padding-left: offset(4)
-        &.sidebar-on-end
-            .section-sidebar
-                @include sidebar(end)
-            @include media-breakpoint-up(desktop)
+            &.sidebar-on-end
+                .section-sidebar
+                    @include sidebar(end)
                 .container
                     .content
                         padding-right: offset(4)
-- 
GitLab