Browse Source

fix: mobile tools positioning (#6107)

* fix: mobile tools positioning

* add var for padding

* use css var

* new line

* stupid mistake

* lint
Aakansha Doshi 2 years ago
parent
commit
99fdffdab7
3 changed files with 9 additions and 6 deletions
  1. 6 4
      src/components/FixedSideContainer.scss
  2. 2 2
      src/css/styles.scss
  3. 1 0
      src/css/theme.scss

+ 6 - 4
src/components/FixedSideContainer.scss

@@ -1,3 +1,5 @@
+@import "../css/variables.module";
+
 .excalidraw {
   .FixedSideContainer {
     position: absolute;
@@ -9,10 +11,10 @@
   }
 
   .FixedSideContainer_side_top {
-    left: 1rem;
-    top: 1rem;
-    right: 1rem;
-    bottom: 1rem;
+    left: var(--editor-container-padding);
+    top: var(--editor-container-padding);
+    right: var(--editor-container-padding);
+    bottom: var(--editor-container-padding);
     z-index: 2;
   }
 

+ 2 - 2
src/css/styles.scss

@@ -541,8 +541,8 @@
 
   .mobile-misc-tools-container {
     position: absolute;
-    top: 5rem;
-    right: 0;
+    top: calc(5rem - var(--editor-container-padding));
+    right: calc(var(--editor-container-padding) * -1);
     display: flex;
     flex-direction: column;
     border: 1px solid var(--sidebar-border-color);

+ 1 - 0
src/css/theme.scss

@@ -42,6 +42,7 @@
   --default-icon-size: 1rem;
   --lg-button-size: 2.25rem;
   --lg-icon-size: 1rem;
+  --editor-container-padding: 1rem;
 
   @media screen and (min-device-width: 1921px) {
     --lg-button-size: 2.5rem;