From 723d6ebf2c4c9fbd13a74bcd70327d87a3f6b1af Mon Sep 17 00:00:00 2001
From: Jialin He <jialin.he@ithaka.org>
Date: Fri, 24 Jan 2025 13:29:37 -0500
Subject: [PATCH] feat(sidenav): change close button style

---
 .changeset/silver-suits-wait.md                            | 5 +++++
 packages/pharos/src/components/sidenav/pharos-sidenav.scss | 2 +-
 packages/pharos/src/components/sidenav/pharos-sidenav.ts   | 2 +-
 3 files changed, 7 insertions(+), 2 deletions(-)
 create mode 100644 .changeset/silver-suits-wait.md

diff --git a/.changeset/silver-suits-wait.md b/.changeset/silver-suits-wait.md
new file mode 100644
index 000000000..5a4684268
--- /dev/null
+++ b/.changeset/silver-suits-wait.md
@@ -0,0 +1,5 @@
+---
+'@ithaka/pharos': patch
+---
+
+Update the close button in sidenav
diff --git a/packages/pharos/src/components/sidenav/pharos-sidenav.scss b/packages/pharos/src/components/sidenav/pharos-sidenav.scss
index e5bb6ce8b..8246f6edb 100644
--- a/packages/pharos/src/components/sidenav/pharos-sidenav.scss
+++ b/packages/pharos/src/components/sidenav/pharos-sidenav.scss
@@ -51,7 +51,7 @@
   opacity: 0;
   position: absolute;
   top: var(--pharos-spacing-one-quarter-x);
-  right: var(--pharos-spacing-one-quarter-x);
+  left: var(--pharos-spacing-one-quarter-x);
 }
 
 :host([has-close-button]) {
diff --git a/packages/pharos/src/components/sidenav/pharos-sidenav.ts b/packages/pharos/src/components/sidenav/pharos-sidenav.ts
index 3d16f2337..d7a68e8a9 100644
--- a/packages/pharos/src/components/sidenav/pharos-sidenav.ts
+++ b/packages/pharos/src/components/sidenav/pharos-sidenav.ts
@@ -133,7 +133,7 @@ export class PharosSidenav extends ScopedRegistryMixin(FocusMixin(SideElement))
     return this.hasCloseButton
       ? html`<pharos-button
           class="side-element__button"
-          icon="chevron-left"
+          icon="close"
           variant="subtle"
           a11y-label="Close menu"
           is-on-background