From d754e7a2ec790cb7b8ea045203285d725c849761 Mon Sep 17 00:00:00 2001
From: Penar Musaraj <pmusaraj@gmail.com>
Date: Mon, 6 Feb 2023 10:04:19 -0500
Subject: [PATCH] FIX: Screen shaking on mobile view navigation (#45)

Setting a border on the `fullscreen-table-wrapper` element causes a
small shake in mobile views, especially on topics with multiple tables.

This PR removes the border on the table entirely. This was an almost
invisible visual effect when selecting the table (small border width,
whole table is set to 50% opacity).

It also makes a small refactor, moving the styles of the button outside
of the hover state.
---
 mobile/mobile.scss                  |  0
 scss/post/table-edit-decorator.scss | 10 ++++------
 2 files changed, 4 insertions(+), 6 deletions(-)
 delete mode 100644 mobile/mobile.scss

diff --git a/mobile/mobile.scss b/mobile/mobile.scss
deleted file mode 100644
index e69de29..0000000
diff --git a/scss/post/table-edit-decorator.scss b/scss/post/table-edit-decorator.scss
index 6cb276b..de9de5a 100644
--- a/scss/post/table-edit-decorator.scss
+++ b/scss/post/table-edit-decorator.scss
@@ -18,23 +18,21 @@
 .mobile-view {
   .btn-edit-table {
     display: none;
+    z-index: 2;
+    position: absolute;
+    top: 1rem;
+    left: 1rem;
   }
 
   .fullscreen-table-wrapper {
     position: relative;
-    border: 1px solid transparent;
 
     &:hover {
-      border: 1px solid var(--primary-low);
       table {
         opacity: 0.5;
       }
       .btn-edit-table {
-        z-index: 2;
         display: block;
-        position: absolute;
-        top: 1rem;
-        left: 1rem;
       }
     }
   }