feat(Slider): the step attribute supports decimals (#1990)

* feat(Slider): the step attribute supports decimals

* fix(slider): resolve emit same value

---------

Co-authored-by: leejimqiu <leejimqiu@gmail.com>

src/image-viewer/__test__/__virtualHostSnapshot__/demo.test.js.snap

@@ -7,7 +7,7 @@ exports[`ImageViewer ImageViewer base demo works fine 1`] = `
     size="large"
     theme="primary"
     variant="outline"
-    bind:tap="clickHandle"
+    bind:tap="onClick"
   >
     基础图片预览
   </t-button>
@@ -34,12 +34,12 @@ exports[`ImageViewer ImageViewer delete demo works fine 1`] = `
     size="large"
     theme="primary"
     variant="outline"
-    bind:tap="clickHandle"
+    bind:tap="onClick"
   >
     带操作图片预览
   </t-button>
-  <t-toast
-    id="t-toast"
+  <t-action-sheet
+    id="t-action-sheet"
   />
   <t-image-viewer
     closeBtn="{{false}}"

src/slider/__test__/__snapshots__/demo.test.js.snap

@@ -4,6 +4,8 @@ exports[`Slider Slider base demo works fine 1`] = `
 <base>
   <t-slider
     defaultValue="{{23}}"
+    step="{{0.1}}"
+    bind:change="handleChange"
   />
 </base>
 `;

src/slider/__test__/__virtualHostSnapshot__/demo.test.js.snap

@@ -4,6 +4,8 @@ exports[`Slider Slider base demo works fine 1`] = `
 <base>
   <t-slider
     defaultValue="{{23}}"
+    step="{{0.1}}"
+    bind:change="handleChange"
   />
 </base>
 `;

src/slider/__test__/index.test.js

@@ -239,8 +239,5 @@ describe('slider', () => {
 
     comp.setData({ value: 101 });
     expect($slider.instance.data._value).toBe(100);
-
-    comp.setData({ value: 'undefined' });
-    expect($slider.instance.data._value).toBe(0);
   });
 });

src/slider/_example/base/index.js

@@ -1 +1,7 @@
-Component({});
+Component({
+  methods: {
+    handleChange(e) {
+      console.log(e.detail.value);
+    },
+  },
+});

src/slider/_example/base/index.wxml

@@ -1 +1 @@
-<t-slider default-value="{{23}}" />
+<t-slider default-value="{{23}}" step="{{0.1}}" bind:change="handleChange" />

src/slider/slider.ts

@@ -121,6 +121,8 @@ export default class Slider extends SuperComponent {
   };
 
   triggerValue(value?: SliderValue) {
+    if (this.preval === value) return;
+    this.preval = value;
     this._trigger('change', {
       value: trimValue(value, this.properties),
     });
@@ -212,11 +214,12 @@ export default class Slider extends SuperComponent {
 
   stepValue(value: number): number {
     const { step, min, max } = this.properties;
-
-    if (Number(step) < 1 || Number(step) > Number(max) - Number(min)) return value;
-
-    const closestStep = trimSingleValue(Math.round(value / Number(step)) * Number(step), Number(min), Number(max));
-
+    const decimal = String(step).indexOf('.') > -1 ? String(step).length - String(step).indexOf('.') - 1 : 0;
+    const closestStep = trimSingleValue(
+      Number((Math.round(value / Number(step)) * Number(step)).toFixed(decimal)),
+      Number(min),
+      Number(max),
+    );
     return closestStep as number;
   }
 
@@ -236,13 +239,12 @@ export default class Slider extends SuperComponent {
     const { pageX } = touch;
     const currentLeft = pageX - initialLeft;
     let value = 0;
-
     if (currentLeft <= 0) {
       value = Number(min);
     } else if (currentLeft >= maxRange) {
       value = Number(max);
     } else {
-      value = Math.round((currentLeft / maxRange) * (Number(max) - Number(min)) + Number(min));
+      value = (currentLeft / maxRange) * (Number(max) - Number(min)) + Number(min);
     }
     return this.stepValue(value);
   }

src/slider/tool.ts

@@ -13,9 +13,6 @@
  * @return {number}
  */
 export const trimSingleValue = (value: any, min: number, max: number): number => {
-  if (typeof value !== 'number') {
-    return min;
-  }
   if (value < min) {
     return min;
   }