refactor(comp: carousel): extract less variable (#703)

IDuxFE · Jan 6, 2022 · 56f1b52 · 56f1b52
1 parent 45a9763
commit 56f1b52
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/packages/components/carousel/style/index.less b/packages/components/carousel/style/index.less
@@ -26,7 +26,7 @@
     .carousel-vertical-center();
 
     &:hover {
-      opacity: @carousel-icon-active-opacity;
+      opacity: @carousel-icon-hover-opacity;
     }
 
     &-prev {
@@ -60,10 +60,11 @@
       }
 
       &-default {
-        width: @carousel-dot-size;
-        height: @carousel-dot-size;
-        border-radius: 50%;
+        width: @carousel-dot-width;
+        height: @carousel-dot-height;
+        border-radius: @carousel-dot-border-radius;
         background-color: @carousel-dot-background-color;
+        outline: none;
       }
     }
 

diff --git a/packages/components/carousel/style/themes/default.variable.less b/packages/components/carousel/style/themes/default.variable.less
@@ -2,10 +2,13 @@
 @carousel-arrow-color: @color-white;
 @carousel-arrow-spacing: @spacing-gutter;
 
-@carousel-dot-size: @spacing-gutter;
+@carousel-dot-width: @spacing-gutter;
+@carousel-dot-height: @spacing-gutter;
 @carousel-dot-background-color: @color-white;
 @carousel-dot-gap: @spacing-md;
 @carousel-dot-spacing: @spacing-lg;
+@carousel-dot-border-radius: 50%;
 
 @carousel-icon-opacity: 0.3;
+@carousel-icon-hover-opacity: 0.6;
 @carousel-icon-active-opacity: 0.8;