fix: set initial browser action popup height in Safari

dessant · Nov 6, 2023 · f738727 · f738727
1 parent 21fbee7
commit f738727
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 2 deletions.
diff --git a/src/action/App.vue b/src/action/App.vue
@@ -705,8 +705,11 @@ body {
   max-height: 100px;
   padding-top: 8px;
   padding-bottom: 24px;
-  transition: max-height 0.3s ease, padding-top 0.3s ease,
-    padding-bottom 0.3s ease, opacity 0.2s ease;
+  transition:
+    max-height 0.3s ease,
+    padding-top 0.3s ease,
+    padding-bottom 0.3s ease,
+    opacity 0.2s ease;
 }
 
 .settings-enter-from,
@@ -758,4 +761,12 @@ html.samsung {
     height: initial;
   }
 }
+
+// Safari 17: the popover opens after a delay the first time the action
+// button is clicked on macOS 14, unless the height is declared.
+@if $target-env == 'safari' {
+  html {
+    min-height: 56px;
+  }
+}
 </style>
diff --git a/webpack.config.js b/webpack.config.js
@@ -127,6 +127,12 @@ module.exports = {
               sassOptions: {
                 includePaths: ['node_modules'],
                 quietDeps: true
+              },
+              additionalData: (content, loaderContext) => {
+                return `
+                  $target-env: "${targetEnv}";
+                  ${content}
+                `;
               }
             }
           }