♿️ Add label to menu button and remove unnecessary tooltip

wancup · Oct 25, 2024 · 587fa13 · 587fa13
1 parent bc1d178
commit 587fa13
Showing 1 changed file with 7 additions and 17 deletions.
diff --git a/src/features/color-theme/components/color-theme-switcher.tsx b/src/features/color-theme/components/color-theme-switcher.tsx
@@ -48,23 +48,13 @@ export function ColorThemeSwitcher(): JSX.Element {
     >
       <Menu.Trigger
         asChild={(triggerProps) => (
-          <Tooltip.Root openDelay={200}>
-            <Tooltip.Trigger>
-              <IconButton {...triggerProps()} variant="ghost" size="sm">
-                {match(theme())
-                  .with("dark", () => <MoonIcon />)
-                  .with("light", () => <SunIcon />)
-                  .with("system", () => <SunMoonIcon />)
-                  .exhaustive()}
-              </IconButton>
-            </Tooltip.Trigger>
-            <Tooltip.Positioner>
-              <Tooltip.Arrow>
-                <Tooltip.ArrowTip />
-              </Tooltip.Arrow>
-              <Tooltip.Content>Change color theme</Tooltip.Content>
-            </Tooltip.Positioner>
-          </Tooltip.Root>
+          <IconButton {...triggerProps()} variant="ghost" size="sm" aria-label="Change theme color">
+            {match(theme())
+              .with("dark", () => <MoonIcon />)
+              .with("light", () => <SunIcon />)
+              .with("system", () => <SunMoonIcon />)
+              .exhaustive()}
+          </IconButton>
         )}
       />
       <Menu.Positioner class={css({ marginRight: "16px" })}>