From 0c18880e5c7f9af6e7a971284c02af815ead3285 Mon Sep 17 00:00:00 2001
From: Cory LaViska <cory@abeautifulsite.net>
Date: Tue, 29 Nov 2022 11:03:58 -0500
Subject: [PATCH] replace icon x => x-lg

---
 src/components/alert/alert.styles.ts  | 2 +-
 src/components/alert/alert.ts         | 2 +-
 src/components/icon/library.system.ts | 6 +++---
 src/components/tab/tab.styles.ts      | 4 ++--
 src/components/tab/tab.ts             | 2 +-
 src/components/tag/tag.styles.ts      | 9 +--------
 src/components/tag/tag.ts             | 2 +-
 7 files changed, 10 insertions(+), 17 deletions(-)

diff --git a/src/components/alert/alert.styles.ts b/src/components/alert/alert.styles.ts
index 590c880c..93e957a8 100644
--- a/src/components/alert/alert.styles.ts
+++ b/src/components/alert/alert.styles.ts
@@ -91,7 +91,7 @@ export default css`
     flex: 0 0 auto;
     display: flex;
     align-items: center;
-    font-size: var(--sl-font-size-large);
+    font-size: var(--sl-font-size-medium);
     padding-inline-end: var(--sl-spacing-medium);
   }
 `;
diff --git a/src/components/alert/alert.ts b/src/components/alert/alert.ts
index e84967b9..26fa0c63 100644
--- a/src/components/alert/alert.ts
+++ b/src/components/alert/alert.ts
@@ -211,7 +211,7 @@ export default class SlAlert extends ShoelaceElement {
                 part="close-button"
                 exportparts="base:close-button__base"
                 class="alert__close-button"
-                name="x"
+                name="x-lg"
                 library="system"
                 label=${this.localize.term('close')}
                 @click=${this.handleCloseClick}
diff --git a/src/components/icon/library.system.ts b/src/components/icon/library.system.ts
index de00b8fc..850e0a2b 100644
--- a/src/components/icon/library.system.ts
+++ b/src/components/icon/library.system.ts
@@ -103,9 +103,9 @@ const icons = {
       <path d="M3.612 15.443c-.386.198-.824-.149-.746-.592l.83-4.73L.173 6.765c-.329-.314-.158-.888.283-.95l4.898-.696L7.538.792c.197-.39.73-.39.927 0l2.184 4.327 4.898.696c.441.062.612.636.282.95l-3.522 3.356.83 4.73c.078.443-.36.79-.746.592L8 13.187l-4.389 2.256z"/>
     </svg>
   `,
-  x: `
-    <svg xmlns="http://www.w3.org/2000/svg" width="16" height="16" fill="currentColor" class="bi bi-x" viewBox="0 0 16 16">
-      <path d="M4.646 4.646a.5.5 0 0 1 .708 0L8 7.293l2.646-2.647a.5.5 0 0 1 .708.708L8.707 8l2.647 2.646a.5.5 0 0 1-.708.708L8 8.707l-2.646 2.647a.5.5 0 0 1-.708-.708L7.293 8 4.646 5.354a.5.5 0 0 1 0-.708z"/>
+  'x-lg': `
+    <svg xmlns="http://www.w3.org/2000/svg" width="16" height="16" fill="currentColor" class="bi bi-x-lg" viewBox="0 0 16 16">
+      <path d="M2.146 2.854a.5.5 0 1 1 .708-.708L8 7.293l5.146-5.147a.5.5 0 0 1 .708.708L8.707 8l5.147 5.146a.5.5 0 0 1-.708.708L8 8.707l-5.146 5.147a.5.5 0 0 1-.708-.708L7.293 8 2.146 2.854Z"/>
     </svg>
   `,
   'x-circle-fill': `
diff --git a/src/components/tab/tab.styles.ts b/src/components/tab/tab.styles.ts
index a60f502e..c43ea8d0 100644
--- a/src/components/tab/tab.styles.ts
+++ b/src/components/tab/tab.styles.ts
@@ -54,8 +54,8 @@ export default css`
   }
 
   .tab__close-button {
-    font-size: var(--sl-font-size-large);
-    margin-inline-start: var(--sl-spacing-2x-small);
+    font-size: var(--sl-font-size-small);
+    margin-inline-start: var(--sl-spacing-small);
   }
 
   .tab__close-button::part(base) {
diff --git a/src/components/tab/tab.ts b/src/components/tab/tab.ts
index 07c5f101..8f43388a 100644
--- a/src/components/tab/tab.ts
+++ b/src/components/tab/tab.ts
@@ -97,7 +97,7 @@ export default class SlTab extends ShoelaceElement {
               <sl-icon-button
                 part="close-button"
                 exportparts="base:close-button__base"
-                name="x"
+                name="x-lg"
                 library="system"
                 label=${this.localize.term('close')}
                 class="tab__close-button"
diff --git a/src/components/tag/tag.styles.ts b/src/components/tag/tag.styles.ts
index 4b0d4dfd..969c5c7b 100644
--- a/src/components/tag/tag.styles.ts
+++ b/src/components/tag/tag.styles.ts
@@ -69,11 +69,6 @@ export default css`
     padding: 0 var(--sl-spacing-x-small);
   }
 
-  .tag--small .tag__remove {
-    margin-inline-start: var(--sl-spacing-2x-small);
-    margin-right: calc(-1 * var(--sl-spacing-3x-small));
-  }
-
   .tag--medium {
     font-size: var(--sl-button-font-size-medium);
     height: calc(var(--sl-input-height-medium) * 0.8);
@@ -91,9 +86,7 @@ export default css`
   }
 
   .tag__remove {
-    font-size: 1.4em;
-    margin-inline-start: var(--sl-spacing-2x-small);
-    margin-inline-end: calc(-1 * var(--sl-spacing-x-small));
+    margin-inline-start: var(--sl-spacing-x-small);
   }
 
   /*
diff --git a/src/components/tag/tag.ts b/src/components/tag/tag.ts
index 7634a43b..0b7fe704 100644
--- a/src/components/tag/tag.ts
+++ b/src/components/tag/tag.ts
@@ -79,7 +79,7 @@ export default class SlTag extends ShoelaceElement {
               <sl-icon-button
                 part="remove-button"
                 exportparts="base:remove-button__base"
-                name="x"
+                name="x-lg"
                 library="system"
                 label=${this.localize.term('remove')}
                 class="tag__remove"