perfect-gui 3.0.5 → 3.0.6

package/README.md

@@ -91,8 +91,7 @@ gui.image('Click this', 'path/to/image', () => {
 
 ```javascript
 // Simple slider, only returns a value to the callback
-// min parameter is optional, default is 0
-// max parameter is optional, default is 1
+// min and max parameters are optional, default is 0 (min) and 1 (max)
 // step parameter is optional, default is (max - min) * 0.01
 gui.slider('Slide this', { value: 5, min: 0, max: 10, step: .1 }, value => {
     console.log('Slider value : ' + value);
@@ -123,6 +122,7 @@ gui.list('Select one', ['apple', 'lime', 'peach'], function(item) {
 <tr><td>vector2</td><td>
 
 ```javascript
+// min and max parameters are optional, default is 0 (min) and 1 (max)
 let folder = gui.vector2('Position', { 
     x: { object: myObject.position, prop: 'x', min: -10, max: 10 },
     y: { object: myObject.position, prop: 'y', min: -10, max: 10 },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "perfect-gui",
-  "version": "3.0.5",
+  "version": "3.0.6",
   "description": "Nice and simple GUI for JavaScript.",
   "main": "src/index.js",
   "directories": {

package/src/index.js

@@ -378,19 +378,16 @@ export default class GUI {
         this._checkMandatoryParams({
             text: 'string',
             data: 'object',
-            minX: 'number',
-            maxX: 'number',
-            minY: 'number',
-            maxY: 'number',
         }, {
             text,
             data,
-            minX: data.x.min,
-            maxX: data.x.max,
-            minY: data.y.min,
-            maxY: data.y.max,
         });
 
+        const minX = data.x.min ?? 0;
+        const maxX = data.x.max ?? 1;
+        const minY = data.y.min ?? 0;
+        const maxY = data.y.max ?? 1;
+
         const objectX = data.x.object;
         const propX = data.x.prop;
         const propXReferenceIndex = this.propReferences.push(objectX[propX]) - 1;
@@ -417,8 +414,8 @@ export default class GUI {
             el: 'div',
             class: 'p-gui__vector2-area',
             onclick: (evt) => {
-                objectX[propX] = parseFloat(this._mapLinear(evt.offsetX, 0, area.clientWidth, data.x.min, data.x.max).toFixed(1));
-                objectY[propY] = parseFloat(this._mapLinear(evt.offsetY, 0, area.clientHeight, data.y.max, data.y.min).toFixed(1));
+                objectX[propX] = parseFloat(this._mapLinear(evt.offsetX, 0, area.clientWidth, minX, maxX).toFixed(1));
+                objectY[propY] = parseFloat(this._mapLinear(evt.offsetY, 0, area.clientHeight, maxY, minY).toFixed(1));
             }
         });
 
@@ -437,13 +434,13 @@ export default class GUI {
             class: 'p-gui__vector2-dot'
         });
 
-        dot.style.left = this._mapLinear(objectX[propX], data.x.min, data.x.max, 0, area.clientWidth) + 'px';
-        dot.style.top = this._mapLinear(objectY[propY], data.y.min, data.y.max, area.clientHeight, 0) + 'px';
+        dot.style.left = this._mapLinear(objectX[propX], minX, maxX, 0, area.clientWidth) + 'px';
+        dot.style.top = this._mapLinear(objectY[propY], minY, maxY, area.clientHeight, 0) + 'px';
 
         Object.defineProperty( objectX, propX, {
             set: val => { 
                 this.propReferences[propXReferenceIndex] = val;
-                dot.style.left = this._mapLinear(val, data.x.min, data.x.max, 0, area.clientWidth) + 'px';
+                dot.style.left = this._mapLinear(val, minX, maxX, 0, area.clientWidth) + 'px';
                 vector_value.textContent = String( val ) + ', ' + objectY[propY];
             },
             get: () => { 
@@ -454,7 +451,7 @@ export default class GUI {
         Object.defineProperty( objectY, propY, {
             set: val => { 
                 this.propReferences[propYReferenceIndex] = val;
-                dot.style.top = this._mapLinear(val, data.y.min, data.y.max, area.clientHeight, 0) + 'px';
+                dot.style.top = this._mapLinear(val, minY, maxY, area.clientHeight, 0) + 'px';
                 vector_value.textContent = objectX[propX] + ', ' + String( val );
             },
             get: () => {