From 3bd78869a0aa817d01f45e177ad8c61bb260d107 Mon Sep 17 00:00:00 2001
From: liyasthomas <liyascthomas@gmail.com>
Date: Thu, 5 Aug 2021 21:29:05 +0530
Subject: [PATCH] fix: responsiveness

---
 assets/scss/styles.scss    | 13 +++++++++++--
 components/app/Footer.vue  |  2 +-
 components/app/Header.vue  |  2 +-
 components/app/Sidenav.vue |  2 +-
 components/smart/Item.vue  |  8 ++++++++
 layouts/default.vue        | 20 +++++++++++++++++---
 pages/settings.vue         |  2 ++
 7 files changed, 41 insertions(+), 8 deletions(-)

diff --git a/assets/scss/styles.scss b/assets/scss/styles.scss
index 07618bd75..6fb7131ed 100644
--- a/assets/scss/styles.scss
+++ b/assets/scss/styles.scss
@@ -105,11 +105,14 @@ a {
 .tippy-popper {
   .tooltip-theme {
     @apply bg-tooltip;
-    @apply text-primary text-xs;
+    @apply text-primary;
     @apply font-semibold;
     @apply py-1 px-2;
     @apply shadow;
 
+    font-size: var(--body-font-size);
+    line-height: var(--body-line-height);
+
     kbd {
       @apply first:ml-2;
       @apply last:-mr-1;
@@ -128,6 +131,9 @@ a {
     @apply p-2;
     @apply shadow-lg;
     @apply focus:outline-none;
+
+    font-size: var(--body-font-size);
+    line-height: var(--body-line-height);
   }
 
   &[x-placement^="top"] .tippy-tooltip .tippy-arrow {
@@ -330,11 +336,14 @@ input[type="checkbox"] {
   .toasted {
     &.toasted-primary {
       @apply bg-tooltip;
-      @apply text-primary text-xs;
+      @apply text-primary;
       @apply justify-start;
       @apply shadow;
       @apply font-semibold;
 
+      font-size: var(--body-font-size);
+      line-height: var(--body-line-height);
+
       .action {
         @apply ml-auto;
         @apply transition;
diff --git a/components/app/Footer.vue b/components/app/Footer.vue
index 3cd28ec3a..43c1a37f1 100644
--- a/components/app/Footer.vue
+++ b/components/app/Footer.vue
@@ -73,7 +73,7 @@
                 @click.native="$refs.options.tippy().hide()"
               />
               <!-- <SmartItem :label="$t('app.status')" /> -->
-              <div class="flex text-xs opacity-50 py-2 px-4">
+              <div class="flex opacity-50 py-2 px-4">
                 {{ `${$t("app.name")} ${$t("app.version")}` }}
               </div>
             </div>
diff --git a/components/app/Header.vue b/components/app/Header.vue
index beb77e67e..7e1e22755 100644
--- a/components/app/Header.vue
+++ b/components/app/Header.vue
@@ -7,7 +7,7 @@
       <span>Hoppscotch</span>
     </div>
     <div class="space-x-2 flex-shrink-0 inline-flex items-center">
-      <AppGitHubStarButton class="mt-1 mr-2" />
+      <AppGitHubStarButton class="flex mx-2 mt-1" />
       <TabPrimary
         id="installPWA"
         v-tippy="{ theme: 'tooltip' }"
diff --git a/components/app/Sidenav.vue b/components/app/Sidenav.vue
index 8e083100d..fd0672b33 100644
--- a/components/app/Sidenav.vue
+++ b/components/app/Sidenav.vue
@@ -1,6 +1,6 @@
 <template>
   <aside>
-    <nav class="flex flex-col flex-nowrap">
+    <nav class="flex flex-nowrap md:flex-col">
       <nuxt-link
         v-for="(navigation, index) in primaryNavigation"
         :key="`navigation-${index}`"
diff --git a/components/smart/Item.vue b/components/smart/Item.vue
index 10acf1792..2cf1b359a 100644
--- a/components/smart/Item.vue
+++ b/components/smart/Item.vue
@@ -19,6 +19,10 @@
       { 'pointer-events-none': loading },
       { 'flex-1': label },
       { 'flex-row-reverse justify-end': reverse },
+      {
+        'border border-divider hover:border-dividerDark focus:border-dividerDark':
+          outline,
+      },
     ]"
     :disabled="disabled"
     :tabindex="loading ? '-1' : '0'"
@@ -99,6 +103,10 @@ export default {
       type: Boolean,
       default: false,
     },
+    outline: {
+      type: Boolean,
+      default: false,
+    },
     infoIcon: {
       type: String,
       default: "",
diff --git a/layouts/default.vue b/layouts/default.vue
index 3445c7ea1..bf5856c1f 100644
--- a/layouts/default.vue
+++ b/layouts/default.vue
@@ -2,11 +2,14 @@
   <div class="flex h-screen w-screen">
     <Splitpanes :dbl-click-splitter="false" horizontal>
       <Pane class="flex flex-1 !overflow-auto">
-        <Splitpanes :dbl-click-splitter="false" vertical>
+        <Splitpanes
+          :dbl-click-splitter="false"
+          :horizontal="!(windowInnerWidth >= 768)"
+        >
           <Pane
             v-if="LEFT_SIDEBAR"
-            style="width: auto"
-            class="hide-scrollbar overflow-auto"
+            style="width: auto; height: auto"
+            class="hide-scrollbar !overflow-auto"
           >
             <AppSidenav />
           </Pane>
@@ -52,6 +55,7 @@ export default defineComponent({
     return {
       LEFT_SIDEBAR: null,
       ZEN_MODE: null,
+      windowInnerWidth: 0,
     }
   },
   subscriptions() {
@@ -83,6 +87,8 @@ export default defineComponent({
     })
   },
   async mounted() {
+    window.addEventListener("resize", this.handleResize)
+    this.handleResize()
     performMigrations()
     console.log(
       "%cWe ❤︎ open source!",
@@ -120,5 +126,13 @@ export default defineComponent({
 
     logPageView(this.$router.currentRoute.fullPath)
   },
+  destroyed() {
+    window.removeEventListener("resize", this.handleResize)
+  },
+  methods: {
+    handleResize() {
+      this.windowInnerWidth = window.innerWidth
+    },
+  },
 })
 </script>
diff --git a/pages/settings.vue b/pages/settings.vue
index 5d211f860..0434e72f5 100644
--- a/pages/settings.vue
+++ b/pages/settings.vue
@@ -250,6 +250,7 @@
                   svg="firefox"
                   label="Firefox"
                   :info-icon="hasFirefoxExtInstalled ? 'check_circle' : ''"
+                  outline
                 />
               </span>
               <span>
@@ -259,6 +260,7 @@
                   svg="chrome"
                   label="Chrome"
                   :info-icon="hasChromeExtInstalled ? 'check_circle' : ''"
+                  outline
                 />
               </span>
             </div>