From 1f68acef8dd386267b9b30c650df6abe11b10954 Mon Sep 17 00:00:00 2001
From: wp-aberg <102534985+wp-aberg@users.noreply.github.com>
Date: Tue, 19 Mar 2024 14:11:23 -0400
Subject: [PATCH] fix: transform select icon on open (#596)

---
 ui/select/src/SelectTrigger.tsx | 14 ++++++++++++--
 ui/theme/src/play.stories.tsx   | 17 +++++++++--------
 2 files changed, 21 insertions(+), 10 deletions(-)

diff --git a/ui/select/src/SelectTrigger.tsx b/ui/select/src/SelectTrigger.tsx
index d852b0296..d987459a5 100644
--- a/ui/select/src/SelectTrigger.tsx
+++ b/ui/select/src/SelectTrigger.tsx
@@ -72,6 +72,16 @@ const IconWrapper = styled("div", {
   gridArea: "icon",
 });
 
+const AnimatedIcon = styled(Icon, {
+  transition: `transform ${theme.transitions.normal} ${theme.transitions.inOut}`,
+  "@reducedMotion": {
+    transition: "none",
+  },
+  '[aria-expanded="true"] &': {
+    transform: "rotate(180deg)",
+  },
+});
+
 type SelectTriggerVariants = WPDS.VariantProps<typeof StyledTrigger>;
 
 type SelectTriggerProps = RadixAccordionTriggerProps &
@@ -127,9 +137,9 @@ export const SelectTrigger = React.forwardRef<
         >
           {children}
           <IconWrapper isDisabled={disabled}>
-            <Icon label="">
+            <AnimatedIcon label="">
               <ChevronDown />
-            </Icon>
+            </AnimatedIcon>
           </IconWrapper>
         </StyledTrigger>
         <SubTextWrapper>
diff --git a/ui/theme/src/play.stories.tsx b/ui/theme/src/play.stories.tsx
index 4316216e6..43795ac28 100644
--- a/ui/theme/src/play.stories.tsx
+++ b/ui/theme/src/play.stories.tsx
@@ -68,12 +68,12 @@ const StyledBox = styled(Box, {
   variants: {
     breakpointActive: {
       true: {
-        backgroundColor: theme.colors.success,
+        backgroundColor: theme.colors.green80,
         color: theme.colors.secondary,
         fontWeight: theme.fontWeights.bold,
       },
       false: {
-        backgroundColor: theme.colors.subtle,
+        backgroundColor: theme.colors.gray300,
       },
     },
   },
@@ -243,7 +243,8 @@ function sleep(ms) {
 }
 
 const activeStyle = `background-color: ${theme.colors.green80.value}`;
-const inactiveStyle = `background-color: ${theme.colors.alpha400.value}`;
+const inactiveStyle = `background-color: ${theme.colors.gray300.value}`;
+const transitionDelay = 500;
 
 const allModes = {
   sm: {
@@ -280,7 +281,7 @@ InteractionsSmall.parameters = {
 
 InteractionsSmall.play = async ({ canvasElement }) => {
   const canvas = within(canvasElement);
-  await sleep(500);
+  await sleep(transitionDelay);
   await expect(canvas.getByText("@sm")).toHaveStyle(activeStyle);
   await expect(canvas.getByText("@md")).toHaveStyle(inactiveStyle);
   await expect(canvas.getByText("@maxSm")).toHaveStyle(activeStyle);
@@ -301,7 +302,7 @@ InteractionsMedium.parameters = {
 
 InteractionsMedium.play = async ({ canvasElement }) => {
   const canvas = within(canvasElement);
-  await sleep(500);
+  await sleep(transitionDelay);
   await expect(canvas.getByText("@md")).toHaveStyle(activeStyle);
   await expect(canvas.getByText("@lg")).toHaveStyle(inactiveStyle);
   await expect(canvas.getByText("@notSm")).toHaveStyle(activeStyle);
@@ -323,7 +324,7 @@ InteractionsLarge.parameters = {
 
 InteractionsLarge.play = async ({ canvasElement }) => {
   const canvas = within(canvasElement);
-  await sleep(500);
+  await sleep(transitionDelay);
   await expect(canvas.getByText("@lg")).toHaveStyle(activeStyle);
   await expect(canvas.getByText("@xl")).toHaveStyle(inactiveStyle);
   await expect(canvas.getByText("@notMd")).toHaveStyle(activeStyle);
@@ -345,7 +346,7 @@ InteractionsExtraLarge.parameters = {
 
 InteractionsExtraLarge.play = async ({ canvasElement }) => {
   const canvas = within(canvasElement);
-  await sleep(500);
+  await sleep(transitionDelay);
   await expect(canvas.getByText("@xl")).toHaveStyle(activeStyle);
   await expect(canvas.getByText("@xxl")).toHaveStyle(inactiveStyle);
   await expect(canvas.getByText("@notLg")).toHaveStyle(activeStyle);
@@ -367,7 +368,7 @@ InteractionsExtraExtraLarge.parameters = {
 
 InteractionsExtraExtraLarge.play = async ({ canvasElement }) => {
   const canvas = within(canvasElement);
-  await sleep(500);
+  await sleep(transitionDelay);
   await expect(canvas.getByText("@xxl")).toHaveStyle(activeStyle);
   await expect(canvas.getByText("@notXxl")).toHaveStyle(inactiveStyle);
   await expect(canvas.getByText("@maxXxl")).toHaveStyle(activeStyle);