fix: remove useAppTheme dependency

f-lab-edu · Aug 10, 2024 · 563931e · 563931e
1 parent ca0a374
commit 563931e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 21 deletions.
diff --git a/packages/design-system/src/components/button/button.module.scss b/packages/design-system/src/components/button/button.module.scss
@@ -14,7 +14,6 @@ $size-map: (
 
 @mixin button-sizes($sizes, $size) {
   width: $size;
-  // height: $size;
   @if ($sizes== "icon") {
     aspect-ratio: 1/1;
   } @else {
@@ -43,12 +42,16 @@ $size-map: (
   }
 }
 
-@mixin theme($theme) {
-  @if ($theme == "light") {
+html[data-theme="light"] {
+  .button {
     background-color: color(default, 50);
     border: 1px solid rgba(0, 0, 0, 0.1);
     color: color(default, 800);
-  } @else if($theme == "dark") {
+  }
+}
+
+html[data-theme="dark"] {
+  .button {
     background-color: color(default, 800);
     border: 1px solid rgba(255, 255, 255, 0.1);
     color: color(default, 50);
@@ -65,12 +68,6 @@ $size-map: (
 }
 
 .button {
-  &.light {
-    @include theme("light");
-  }
-  &.dark {
-    @include theme("dark");
-  }
   align-items: center;
   border-radius: shape("sm");
   box-sizing: border-box;
@@ -101,12 +98,7 @@ $size-map: (
 
 .tooltip-button {
   box-sizing: border-box;
-  &.light {
-    @include theme("light");
-  }
-  &.dark {
-    @include theme("dark");
-  }
+
   position: relative;
   outline: none;
 }

diff --git a/packages/design-system/src/components/button/button.tsx b/packages/design-system/src/components/button/button.tsx
@@ -17,12 +17,8 @@ const Button = ({
   size = "m",
   ...others
 }: ButtonProps) => {
-  const { mode } = useAppTheme();
   return (
-    <button
-      className={style(className, "button", mode == null ? "os" : mode, size)}
-      {...others}
-    >
+    <button className={style(className, "button", size)} {...others}>
       {children}
     </button>
   );