From 40b83b92039458411d641755571f08f29ac30df8 Mon Sep 17 00:00:00 2001
From: Zack Krida <zackkrida@pm.me>
Date: Mon, 11 Apr 2022 16:37:58 -0400
Subject: [PATCH] Fix app layout issues

---
 src/components/VFilters/VSearchGridFilter.vue | 16 +-------------
 src/components/VHeader/VSidebarContent.vue    |  6 +++---
 src/layouts/default.vue                       | 21 ++++++-------------
 3 files changed, 10 insertions(+), 33 deletions(-)

diff --git a/src/components/VFilters/VSearchGridFilter.vue b/src/components/VFilters/VSearchGridFilter.vue
index 8ac4d29988..a93997700c 100644
--- a/src/components/VFilters/VSearchGridFilter.vue
+++ b/src/components/VFilters/VSearchGridFilter.vue
@@ -1,8 +1,5 @@
 <template>
-  <div
-    class="filters py-8 px-10 min-h-full md:bg-dark-charcoal-06"
-    data-testid="filters-list"
-  >
+  <div class="filters py-8 px-10" data-testid="filters-list">
     <div class="flex items-center justify-between mt-2 mb-6">
       <h4 class="text-sr font-semibold py-2 uppercase">
         {{ $t('filter-list.filter-by') }}
@@ -108,14 +105,3 @@ export default {
   },
 }
 </script>
-<style scoped>
-.filters {
-  border-inline-start: 1px solid transparent;
-}
-@screen md {
-  .filters {
-    /* dark-charcoal-20*/
-    border-inline-start: 1px solid #d6d4d5;
-  }
-}
-</style>
diff --git a/src/components/VHeader/VSidebarContent.vue b/src/components/VHeader/VSidebarContent.vue
index 39c99d88d4..d8ce2611d0 100644
--- a/src/components/VHeader/VSidebarContent.vue
+++ b/src/components/VHeader/VSidebarContent.vue
@@ -1,7 +1,7 @@
 <template>
-  <VTeleport v-if="visible" :to="to"
-    ><VSearchGridFilter @close="$emit('close')"
-  /></VTeleport>
+  <VTeleport v-if="visible" :to="to">
+    <VSearchGridFilter @close="$emit('close')" />
+  </VTeleport>
 </template>
 
 <script>
diff --git a/src/layouts/default.vue b/src/layouts/default.vue
index 9f899385b0..72f33814aa 100644
--- a/src/layouts/default.vue
+++ b/src/layouts/default.vue
@@ -1,17 +1,16 @@
 <template>
-  <div class="app grid h-screen overflow-hidden relative">
-    <div>
+  <div class="app grid relative">
+    <div class="sticky top-0 block z-40">
       <VTeleportTarget name="skip-to-content" :force-destroy="true" />
       <VMigrationNotice v-show="isReferredFromCc" />
       <VTranslationStatusBanner />
       <VHeader />
     </div>
-    <main
-      class="main embedded overflow-x-hidden"
-      :class="{ 'has-sidebar': isSidebarVisible }"
-    >
+    <main class="main embedded" :class="{ 'has-sidebar': isSidebarVisible }">
       <Nuxt ref="mainContentRef" class="min-w-0 main-page" />
-      <VSidebarTarget class="sidebar" />
+      <VSidebarTarget
+        class="sidebar fixed pb-20 right-0 [calc(100%-81px)] bg-dark-charcoal-06 border-s border-dark-charcoal-20 overflow-y-scroll"
+      />
     </main>
     <VModalTarget class="modal" />
     <VGlobalAudioSection />
@@ -121,12 +120,4 @@ export default embeddedPage
     grid-column: 1;
   }
 }
-
-.main {
-  overflow: hidden;
-}
-.main > *:not(:empty) {
-  overflow-y: scroll;
-  height: 100%;
-}
 </style>