layout fixes

viamrobotics · Jul 24, 2023 · 686214a · 686214a
1 parent efc1c27
commit 686214a
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 23 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@viamrobotics/prime",
-  "version": "0.3.5",
+  "version": "0.3.6",
   "license": "Apache-2.0",
   "type": "module",
   "files": [

diff --git a/src/elements/button/button.svelte b/src/elements/button/button.svelte
@@ -47,14 +47,14 @@ const handleParentClick = (event: PointerEvent) => {
   event.stopImmediatePropagation();
 };
 
-let fill = 'danger';
+let fill = 'black';
 
 $: {
   switch (variant) {
     case 'inverse-primary':
     case 'danger':
     case 'success': {
-      fill = 'reversed';
+      fill = 'white';
       break;
     }
     case 'outline-danger':

diff --git a/src/elements/icon/icon.svelte b/src/elements/icon/icon.svelte
@@ -27,15 +27,17 @@ export let size: Size = 'base';
 const hasNameProperty = Object.hasOwn(paths, name);
 </script>
 
-<svg
-  class={cx(`leading ${sizes[size]}`, {
-    'inline-block': hasNameProperty,
-    hidden: !hasNameProperty,
-  })}
-  viewBox="0 0 24 24"
-  aria-labelledby="name"
-  role="img"
->
-  <title id={name}>{`${name} icon`}</title>
-  <path d={paths[name]} fill={fillColor[fill]} />
-</svg>
+<div class="leading-3">
+  <svg
+    class={cx(`${sizes[size]}`, {
+      'inline-block': hasNameProperty,
+      hidden: !hasNameProperty,
+    })}
+    viewBox="0 0 24 24"
+    aria-labelledby="name"
+    role="img"
+  >
+    <title id={name}>{`${name} icon`}</title>
+    <path d={paths[name]} fill={fillColor[fill]} />
+  </svg>
+</div>
diff --git a/src/elements/select/select.svelte b/src/elements/select/select.svelte
@@ -275,14 +275,7 @@ $: {
 
     {#if tooltip}
       <v-tooltip text={tooltip}>
-        <div
-          class={cx({
-            'text-warning-bright': state === 'warn',
-            'text-danger-dark': state === 'error',
-          })}
-        >
-          <v-icon {fill} name={icon} />
-        </div>
+        <v-icon {fill} name={icon} />
       </v-tooltip>
     {/if}
   </div>