minitel-standalone - npm Package Compare versions

Comparing version 1.7.0 to 1.7.1

dist/abstract/minitelobject.js

@@ -22,4 +22,6 @@ "use strict";
             set: (function (target, prop, val) {
+                const oldTP = target[prop];
                 target[prop] = val;
-                this.minitel.invalidateRender();
+                if (val !== oldTP)
+                    this.minitel.invalidateRender();
                 return true;
@@ -26,0 +28,0 @@ }).bind(this),

dist/components/input.js

@@ -22,4 +22,6 @@ "use strict";
     set value(newValue) {
+        const oldValue = this._value;
         this._value = newValue;
-        this.minitel.invalidateRender();
+        if (newValue !== oldValue)
+            this.minitel.invalidateRender();
     }
@@ -30,4 +32,6 @@ get value() {
     set cursorActuallyAt(newPos) {
+        const oldValue = this._cursorActuallyAt;
         this._cursorActuallyAt = [newPos[0], newPos[1]];
-        this.minitel.invalidateRender();
+        if (oldValue[0] !== newPos[0] || oldValue[1] !== newPos[1])
+            this.minitel.invalidateRender();
     }
@@ -38,4 +42,6 @@ get cursorActuallyAt() {
     set scrollDelta(newDelta) {
+        const oldDelta = [this._scrollDelta[0], this._scrollDelta[1]];
         this._scrollDelta = [newDelta[0], newDelta[1]];
-        this.minitel.invalidateRender();
+        if (oldDelta[0] !== newDelta[0] || oldDelta[1] !== newDelta[1])
+            this.minitel.invalidateRender();
     }
@@ -46,10 +52,10 @@ get scrollDelta() {
     set focused(val) {
+        if (this._focused !== val)
+            this.minitel.invalidateRender();
         if (val) {
             if (this.minitel.focusedObj)
                 this.minitel.focusedObj.focused = false;
-            this.minitel.invalidateRender();
             this._focused = true;
         }
         else {
-            this.minitel.invalidateRender();
             this._focused = false;
@@ -194,2 +200,3 @@ }
             attributes.onScroll([...this.scrollDelta]);
+            this.oldScrollDelta = [this.scrollDelta[0], this.scrollDelta[1]];
         }
@@ -196,0 +203,0 @@ return result;

dist/components/scrollable.js

@@ -17,10 +17,10 @@ "use strict";
     set focused(val) {
+        if (this._focused !== val)
+            this.minitel.invalidateRender();
         if (val) {
             if (this.minitel.focusedObj)
                 this.minitel.focusedObj.focused = false;
-            this.minitel.invalidateRender();
             this._focused = true;
         }
         else {
-            this.minitel.invalidateRender();
             this._focused = false;
@@ -27,0 +27,0 @@ }

package.json

 {
   "name": "minitel-standalone",
-  "version": "1.7.0",
+  "version": "1.7.1",
   "description": "A standalone package for minitel components",
@@ -5,0 +5,0 @@ "main": "dist/index.js",

src/abstract/minitelobject.ts

@@ -37,4 +37,5 @@ import { EventEmitter } from 'node:events';
             set: (function (this: MinitelObject, target: Record<string | symbol, any>, prop: string | symbol, val: any) {
+                const oldTP = target[prop];
                 target[prop] = val;
-                this.minitel.invalidateRender();
+                if (val !== oldTP) this.minitel.invalidateRender();
                 return true;
@@ -41,0 +42,0 @@ }).bind(this),

src/components/input.ts

@@ -43,4 +43,5 @@ import { Focusable, FocusableAttributes } from '../abstract/focusable.js';
     set value(newValue: string) {
+        const oldValue = this._value;
         this._value = newValue;
-        this.minitel.invalidateRender();
+        if (newValue !== oldValue) this.minitel.invalidateRender();
     }
@@ -51,4 +52,5 @@ get value() {
     set cursorActuallyAt(newPos: [number, number]) {
+        const oldValue = this._cursorActuallyAt;
         this._cursorActuallyAt = [newPos[0], newPos[1]];
-        this.minitel.invalidateRender();
+        if (oldValue[0] !== newPos[0] || oldValue[1] !== newPos[1]) this.minitel.invalidateRender();
     }
@@ -59,4 +61,5 @@ get cursorActuallyAt() {
     set scrollDelta(newDelta: [number, number]) {
+        const oldDelta = [this._scrollDelta[0], this._scrollDelta[1]];
         this._scrollDelta = [newDelta[0], newDelta[1]];
-        this.minitel.invalidateRender();
+        if (oldDelta[0] !== newDelta[0] || oldDelta[1] !== newDelta[1]) this.minitel.invalidateRender();
     }
@@ -67,8 +70,7 @@ get scrollDelta() {
     set focused(val) {
+        if (this._focused !== val) this.minitel.invalidateRender();
         if (val) {
             if (this.minitel.focusedObj) this.minitel.focusedObj.focused = false;
-            this.minitel.invalidateRender();
             this._focused = true;
         } else {
-            this.minitel.invalidateRender();
             this._focused = false;
@@ -229,2 +231,3 @@ }
             attributes.onScroll([...this.scrollDelta]);
+            this.oldScrollDelta = [this.scrollDelta[0], this.scrollDelta[1]];
         }
@@ -231,0 +234,0 @@

src/components/scrollable.ts

@@ -38,8 +38,7 @@ import { Focusable } from '../abstract/focusable.js';
     set focused(val) {
+        if (this._focused !== val) this.minitel.invalidateRender();
         if (val) {
             if (this.minitel.focusedObj) this.minitel.focusedObj.focused = false;
-            this.minitel.invalidateRender();
             this._focused = true;
         } else {
-            this.minitel.invalidateRender();
             this._focused = false;
@@ -46,0 +45,0 @@ }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

187935

increased by0.48%

Lines of code

4107

increased by0.29%

No dependency changes