@refinitiv-ui/elements 6.10.0 → 6.10.2

package/CHANGELOG.md

@@ -3,6 +3,19 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [6.10.2](https://github.com/Refinitiv/refinitiv-ui/compare/@refinitiv-ui/elements@6.10.1...@refinitiv-ui/elements@6.10.2) (2023-09-04)
+
+### Bug Fixes
+
+- **number-field:** stepUp/stepDown not fire input event ([#925](https://github.com/Refinitiv/refinitiv-ui/issues/925)) ([cb2b994](https://github.com/Refinitiv/refinitiv-ui/commit/cb2b994dbdb0adf520f95f2a1239eab55024106f))
+
+## [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/number-field/custom-elements.json

@@ -158,6 +158,9 @@
         {
           "name": "error-changed",
           "description": "Fired when user inputs invalid value. The event is not triggered if `error` property is changed programmatically."
+        },
+        {
+          "name": "input"
         }
       ],
       "methods": [

package/lib/number-field/custom-elements.md

@@ -33,4 +33,5 @@ Form control element for numbers.
 | Event           | Description                                      |
 |-----------------|--------------------------------------------------|
 | `error-changed` | Fired when user inputs invalid value. The event is not triggered if `error` property is changed programmatically. |
+| `input`         |                                                  |
 | `value-changed` | Fired when user commits a value change. The event is not triggered if `value` property is changed programmatically. |

package/lib/number-field/index.js

@@ -278,6 +278,7 @@ let NumberField = class NumberField extends FormFieldElement {
     onApplyStep(direction) {
         try {
             this.applyStepDirection(direction);
+            this.dispatchEvent(new InputEvent('input'));
             this.setSilentlyValueAndNotify();
         }
         catch (error) {

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}
     `;
     }
     /**
@@ -1223,7 +1213,6 @@ let Slider = class Slider extends ControlElement {
         @blur=${this.onNumberFieldBlur}
         @keydown=${this.onNumberFieldKeyDown}
         @input=${this.onNumberFieldInput}
-        @value-changed=${this.onNumberFieldInput}
         part="input"
         name="${name}"
         no-spinner

package/lib/version.js

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

package/package.json

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