@refinitiv-ui/elements 6.10.0 → 6.10.1

package/CHANGELOG.md

@@ -3,6 +3,13 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [6.10.1](https://github.com/Refinitiv/refinitiv-ui/compare/@refinitiv-ui/elements@6.10.0...@refinitiv-ui/elements@6.10.1) (2023-08-28)
+
+### Bug Fixes
+
+- **slider:** display and initial to value incorrectly ([#914](https://github.com/Refinitiv/refinitiv-ui/issues/914)) ([f1964db](https://github.com/Refinitiv/refinitiv-ui/commit/f1964dbb42721ca84efe4c79014b43573a289a11))
+- **slider:** page freeze when from/to overflow max value ([#920](https://github.com/Refinitiv/refinitiv-ui/issues/920)) ([efa8314](https://github.com/Refinitiv/refinitiv-ui/commit/efa8314cda00e3a1c4d31bb24e50e46b835ba95d))
+
 # [6.10.0](https://github.com/Refinitiv/refinitiv-ui/compare/@refinitiv-ui/elements@6.9.2...@refinitiv-ui/elements@6.10.0) (2023-08-21)
 
 ### Bug Fixes

package/lib/slider/index.js

@@ -445,8 +445,8 @@ let Slider = class Slider extends ControlElement {
                     }
                 }
                 else {
-                    this.from = clamp(this.fromNumber || this.minNumber, this.minNumber, this.toNumber);
-                    this.to = clamp(this.toNumber || this.maxNumber, this.fromNumber, this.maxNumber);
+                    this.from = clamp(this.fromNumber, this.minNumber, this.toNumber);
+                    this.to = clamp(this.toNumber, this.fromNumber, this.maxNumber);
                 }
             }
             else {
@@ -966,10 +966,8 @@ let Slider = class Slider extends ControlElement {
         }
         else {
             // if value is outside boundary, set to boundary
-            if (this.fromNumber < this.minNumber) {
-                this.from = this.min;
-            }
-            else if (this.fromNumber > this.toNumber) {
+            this.from = clamp(this.fromNumber, this.minNumber, this.maxNumber);
+            if (this.fromNumber > this.toNumber) {
                 this.from = this.to;
             }
             if (this.minRangeNumber) {
@@ -991,24 +989,19 @@ let Slider = class Slider extends ControlElement {
      * @returns true if value and step inside a boundary
      */
     isValueInBoundary(value, valueFor) {
-        if (this.minNumber < this.maxNumber) {
-            // Check if value is in range
-            if (this.range) {
-                if (valueFor === SliderDataName.to) {
-                    if (value < this.fromNumber + this.minRangeNumber || value > this.maxNumber) {
-                        return false;
-                    }
-                }
-                else if (value < this.minNumber || value > this.toNumber - this.minRangeNumber) {
-                    return false;
-                }
-            }
-            else if (value < this.minNumber || value > this.maxNumber) {
+        if (this.minNumber > this.maxNumber) {
+            return false;
+        }
+        // Check if value is in range
+        if (value < this.minNumber || value > this.maxNumber) {
+            return false;
+        }
+        if (this.range) {
+            if (valueFor === SliderDataName.to && value < this.fromNumber + this.minRangeNumber) {
                 return false;
             }
-            // check step min and max in range
-            if (this.stepRange < this.minNumber || this.stepRange > this.maxNumber) {
-                return true;
+            else if (value > this.toNumber - this.minRangeNumber) {
+                return false;
             }
         }
         return true;
@@ -1023,12 +1016,10 @@ let Slider = class Slider extends ControlElement {
         }
         else {
             // if value is outside boundary, set to boundary
+            this.to = clamp(this.toNumber, this.minNumber, this.maxNumber);
             if (this.toNumber < this.fromNumber) {
                 this.to = this.from;
             }
-            else if (this.toNumber > this.maxNumber) {
-                this.to = this.max;
-            }
             if (this.minRangeNumber) {
                 const distanceFromTo = Math.abs(this.toNumber - this.fromNumber);
                 const distanceMax = this.fromNumber + this.minRangeNumber;
@@ -1055,9 +1046,8 @@ let Slider = class Slider extends ControlElement {
     onMinRangeChange() {
         const valueMinRange = Math.abs(this.minRangeNumber);
         const maximumRangeMinMax = Math.abs(this.maxNumber - this.minNumber);
-        const maximumRangeFromTo = Math.abs(this.toNumber - this.fromNumber);
         if (valueMinRange && valueMinRange >= this.stepNumber) {
-            if (valueMinRange <= maximumRangeMinMax && valueMinRange <= maximumRangeFromTo) {
+            if (valueMinRange <= maximumRangeMinMax) {
                 this.minRange = valueMinRange.toString();
             }
             else {
@@ -1201,8 +1191,8 @@ let Slider = class Slider extends ControlElement {
      */
     renderThumb(from, to) {
         return html `
-      ${this.thumbTemplate(from, this.calculatePosition(from), to ? SliderDataName.from : SliderDataName.value)}
-      ${to && this.thumbTemplate(to, this.calculatePosition(to), SliderDataName.to)}
+      ${this.thumbTemplate(from, this.calculatePosition(from), to !== undefined ? SliderDataName.from : SliderDataName.value)}
+      ${to !== undefined ? this.thumbTemplate(to, this.calculatePosition(to), SliderDataName.to) : nothing}
     `;
     }
     /**

package/lib/version.js

@@ -1 +1 @@
-export const VERSION = '6.10.0';
+export const VERSION = '6.10.1';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@refinitiv-ui/elements",
-  "version": "6.10.0",
+  "version": "6.10.1",
   "description": "Element Framework Elements",
   "author": "Refinitiv",
   "license": "Apache-2.0",
@@ -369,5 +369,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "5df63d07a3a62aa3b4bb613a67f533a053eca260"
+  "gitHead": "14f4b70dbd8dc19e883d3b646981bf5f43724e20"
 }