mapscreenr - npm Package Compare versions

Comparing version 0.6.2 to 0.7.0

package.json

 {
-  "name": "mapscreenr",
-  "description": "A flexible container for map attributes and viewport.",
-  "version": "0.6.2",
-  "author": {
-    "name": "Josh Goldberg",
-    "email": "[e-mail]"
-  },
-  "repository": {
-    "type": "git",
-    "url": "ssh://[e-mail]:FullScreenShenanigans/MapScreenr.git"
-  },
-  "bugs": {
-    "url": "https://github.com/FullScreenShenanigans/MapScreenr/issues"
-  },
-  "license": "MIT",
-  "devDependencies": {
-    "gulp-shenanigans": "0.6.X"
-  }
+    "author": {
+        "email": "[e-mail]",
+        "name": "Josh Goldberg"
+    },
+    "bugs": {
+        "url": "https://github.com/FullScreenShenanigans/MapScreenr/issues"
+    },
+    "dependencies": {},
+    "description": "A flexible container for map attributes and viewport.",
+    "devDependencies": {
+        "@types/chai": "^4.0.4",
+        "@types/lolex": "^1.5.32",
+        "@types/mocha": "^2.2.44",
+        "@types/sinon": "^4.0.0",
+        "@types/sinon-chai": "^2.7.29",
+        "chai": "^4.1.2",
+        "glob": "^7.1.2",
+        "lolex": "^2.3.0",
+        "mocha": "^4.0.1",
+        "mocha-headless-chrome": "^1.7.1",
+        "requirejs": "^2.3.5",
+        "run-for-every-file": "^1.1.0",
+        "shenanigans-manager": "^0.2.0",
+        "sinon": "^4.1.2",
+        "sinon-chai": "^2.14.0",
+        "tslint": "5.8.0",
+        "typedoc": "^0.9.0",
+        "typescript": "^2.6.2",
+        "webpack": "^3.10.0"
+    },
+    "license": "MIT",
+    "name": "mapscreenr",
+    "repository": {
+        "type": "git",
+        "url": "ssh://[e-mail]:FullScreenShenanigans/MapScreenr.git"
+    },
+    "scripts": {
+        "dist": "npm run dist:webpack",
+        "dist:webpack": "webpack --config webpack.config.js",
+        "docs": "npm run docs:typedoc",
+        "docs:typedoc": "typedoc src/ --out docs/generated",
+        "init": "npm install && npm run setup && npm run verify",
+        "setup": "npm run setup:copy && npm run setup:package",
+        "setup:copy": "npm run setup:copy:default",
+        "setup:copy:default": "run-for-every-file --dot --src \"node_modules/shenanigans-manager/setup/default/\" --file \"**/*\" --run \"mustache package.json {{src-file}} {{file}}\" --dest \".\" --only-files",
+        "setup:package": "shenanigans-manager hydrate-package-json",
+        "src": "npm run src:tsc && npm run src:tslint",
+        "src:tsc": "tsc -p .",
+        "src:tslint": "tslint -c tslint.json --exclude ./node_modules/**/*.ts* -p tsconfig.json -t stylish",
+        "test": "npm run test:setup && npm run test:run",
+        "test:run": "mocha-headless-chrome --file test/index.html",
+        "test:setup": "npm run test:setup:copy && npm run test:setup:html && npm run test:setup:tsc",
+        "test:setup:copy": "npm run test:setup:copy:default",
+        "test:setup:copy:default": "run-for-every-file --dot --src \"node_modules/shenanigans-manager/setup/test/\" --file \"**/*\" --run \"mustache package.json {{src-file}} ./test/{{file}}\" --dest \".\" --only-files",
+        "test:setup:html": "shenanigans-manager generate-test-html",
+        "test:setup:tsc": "tsc -p test",
+        "verify": "npm run src && npm run test && npm run dist && npm run docs",
+        "watch": "run-p \"tsc -p . -w\" \"echo sup\""
+    },
+    "shenanigans": {
+        "name": "MapScreenr"
+    },
+    "types": "./src/index.d.ts",
+    "version": "0.7.0"
 }

README.md

@@ -0,1 +1,2 @@
+<!-- {{Top}} -->
 # MapScreenr
@@ -6,17 +7,10 @@ [![Build Status](https://travis-ci.org/FullScreenShenanigans/MapScreenr.svg?branch=master)](https://travis-ci.org/FullScreenShenanigans/MapScreenr)
 A flexible container for map attributes and viewport.
+<!-- {{/Top}} -->
 
+<!-- {{Development}} -->
+## Development
 
+See [Documentation/Development](https://github.com/FullScreenShenanigans/Documentation).
 
-## Build Process
 
-MapScreenr uses [Gulp](http://gulpjs.com/) to automate building, which requires [Node.js](http://node.js.org).
-
-To build from scratch, install NodeJS and run the following commands:
-
-```
-npm install -g gulp
-npm install
-gulp
-```
-
-See [Build Details](https://github.com/FullScreenShenanigans/Documentation/blob/master/Build%20Details.md) for detailed Gulp usage.
+<!-- {{/Development}} -->

src/IMapScreenr.d.ts

 /**
- * Functions to compute new variable values, keyed by their variable's names.
+ * functions to compute new variable values, keyed by their variable's names.
  */
 export interface IVariableFunctions {
-    [i: string]: Function;
+    [i: string]: () => any;
 }
@@ -31,6 +31,2 @@ /**
     /**
-     * Arguments to be passed to variable Functions.
-     */
-    variableArgs?: any[];
-    /**
      * Assorted known variables, keyed by name.
@@ -77,10 +73,6 @@ */
     /**
-     * A listing of variable Functions to be calculated on screen resets.
+     * A listing of variable functions to be calculated on screen resets.
      */
     variableFunctions: IVariableFunctions;
     /**
-     * Arguments to be passed into variable computation Functions.
-     */
-    variableArgs: any[];
-    /**
      * Known variables, keyed by name.
@@ -103,10 +95,10 @@ */
     /**
-     * Recalculates all variables by passing variableArgs to their Functions.
+     * Recalculates all variables.
      */
     setVariables(): void;
     /**
-     * Recalculates a variable by passing variableArgs to its Function.
+     * Recalculates a variable by passing variableArgs to its function.
      *
      * @param name   The name of the variable to recalculate.
-     * @param value   A new value for the variable instead of its Function's result.
+     * @param value   A new value for the variable instead of its function's result.
      * @returns The new value of the variable.
@@ -113,0 +105,0 @@ */

src/IMapScreenr.js

 define(["require", "exports"], function (require, exports) {
     "use strict";
+    Object.defineProperty(exports, "__esModule", { value: true });
 });
-
-//# sourceMappingURL=data:application/json;charset=utf8;base64,eyJ2ZXJzaW9uIjozLCJzb3VyY2VzIjpbXSwibmFtZXMiOltdLCJtYXBwaW5ncyI6IiIsImZpbGUiOiJJTWFwU2NyZWVuci5qcyIsInNvdXJjZXNDb250ZW50IjpbXX0=

src/IMapScreenr.ts

 /**
- * Functions to compute new variable values, keyed by their variable's names.
+ * functions to compute new variable values, keyed by their variable's names.
  */
 export interface IVariableFunctions {
-    [i: string]: Function;
+    [i: string]: () => any;
 }
@@ -36,7 +36,2 @@
     /**
-     * Arguments to be passed to variable Functions.
-     */
-    variableArgs?: any[];
-
-    /**
      * Assorted known variables, keyed by name.
@@ -92,3 +87,3 @@ */
     /**
-     * A listing of variable Functions to be calculated on screen resets.
+     * A listing of variable functions to be calculated on screen resets.
      */
@@ -98,7 +93,2 @@ variableFunctions: IVariableFunctions;
     /**
-     * Arguments to be passed into variable computation Functions.
-     */
-    variableArgs: any[];
-
-    /**
      * Known variables, keyed by name.
@@ -125,3 +115,3 @@ */
     /**
-     * Recalculates all variables by passing variableArgs to their Functions.
+     * Recalculates all variables.
      */
@@ -131,6 +121,6 @@ setVariables(): void;
     /**
-     * Recalculates a variable by passing variableArgs to its Function.
-     * 
+     * Recalculates a variable by passing variableArgs to its function.
+     *
      * @param name   The name of the variable to recalculate.
-     * @param value   A new value for the variable instead of its Function's result.
+     * @param value   A new value for the variable instead of its function's result.
      * @returns The new value of the variable.
@@ -142,3 +132,3 @@ */
      * Shifts the MapScreenr horizontally and vertically via shiftX and shiftY.
-     * 
+     *
      * @param dx   How far to scroll horizontally.
@@ -151,3 +141,3 @@ * @param dy   How far to scroll vertically.
      * Shifts the MapScreenr horizontally by changing left and right by the dx.
-     * 
+     *
      * @param dx   How far to scroll horizontally.
@@ -159,3 +149,3 @@ */
      * Shifts the MapScreenr vertically by changing top and bottom by the dy.
-     * 
+     *
      * @param dy   How far to scroll vertically.
@@ -162,0 +152,0 @@ */

src/MapScreenr.d.ts

@@ -9,8 +9,4 @@ import { IMapScreenr, IMapScreenrSettings, IVariableFunctions, IVariables } from "./IMapScreenr";
      */
-    variableFunctions: IVariableFunctions;
+    readonly variableFunctions: IVariableFunctions;
     /**
-     * Arguments to be passed into variable computation Functions.
-     */
-    variableArgs: any[];
-    /**
      * Top border measurement of the bounding box.
@@ -42,16 +38,15 @@ */
      */
-    width: number;
+    readonly width: number;
     /**
      * Constant height of the bounding box.
      */
-    height: number;
+    readonly height: number;
     /**
      * Assorted known variables, keyed by name.
      */
-    variables: IVariables;
+    readonly variables: IVariables;
     /**
-     * Resets the MapScreenr. All members of the settings argument are copied
-     * to the MapScreenr itself, though only width and height are required.
+     * Initializes a new instance of the MapScreenr class.
      *
-     * @param {IMapScreenrSettings} settings
+     * @param settings   Settings to be used for initialization.
      */
@@ -73,3 +68,3 @@ constructor(settings: IMapScreenrSettings);
     /**
-     * Recalculates all variables by passing variableArgs to their Functions.
+     * Recalculates all variables.
      */
@@ -76,0 +71,0 @@ setVariables(): void;

src/MapScreenr.js

 define(["require", "exports"], function (require, exports) {
     "use strict";
+    Object.defineProperty(exports, "__esModule", { value: true });
     /**
      * A flexible container for map attributes and viewport.
      */
-    var MapScreenr = (function () {
+    var MapScreenr = /** @class */ (function () {
         /**
-         * Resets the MapScreenr. All members of the settings argument are copied
-         * to the MapScreenr itself, though only width and height are required.
+         * Initializes a new instance of the MapScreenr class.
          *
-         * @param {IMapScreenrSettings} settings
+         * @param settings   Settings to be used for initialization.
          */
@@ -18,14 +18,5 @@ function MapScreenr(settings) {
             this.variables = {};
-            if (typeof settings === "undefined") {
-                throw new Error("No settings object given to MapScreenr.");
-            }
-            if (!settings.width) {
-                throw new Error("No width given to MapScreenr.");
-            }
-            if (!settings.height) {
-                throw new Error("No height given to MapScreenr.");
-            }
-            if (settings.variables) {
+            if (settings.variables !== undefined) {
                 for (var name_1 in settings.variables) {
-                    if (settings.variables.hasOwnProperty(name_1)) {
+                    if ({}.hasOwnProperty.call(settings.variables, name_1)) {
                         this.variables[name_1] = settings.variables[name_1];
@@ -37,4 +28,5 @@ }
             this.width = settings.width;
-            this.variableFunctions = settings.variableFunctions || {};
-            this.variableArgs = settings.variableArgs || [];
+            this.variableFunctions = settings.variableFunctions === undefined
+                ? {}
+                : settings.variableFunctions;
         }
@@ -67,9 +59,7 @@ /**
         /**
-         * Recalculates all variables by passing variableArgs to their Functions.
+         * Recalculates all variables.
          */
         MapScreenr.prototype.setVariables = function () {
             for (var i in this.variableFunctions) {
-                if (this.variableFunctions.hasOwnProperty(i)) {
-                    this.setVariable(i);
-                }
+                this.setVariable(i);
             }
@@ -86,3 +76,3 @@ };
             this.variables[name] = arguments.length === 1
-                ? this.variableFunctions[name].apply(this, this.variableArgs)
+                ? this.variableFunctions[name]()
                 : value;
@@ -97,6 +87,6 @@ };
         MapScreenr.prototype.shift = function (dx, dy) {
-            if (dx) {
+            if (dx !== 0) {
                 this.shiftX(dx);
             }
-            if (dy) {
+            if (dy !== 0) {
                 this.shiftY(dy);
@@ -127,3 +117,1 @@ }
 });
-
-//# sourceMappingURL=data:application/json;charset=utf8;base64,eyJ2ZXJzaW9uIjozLCJzb3VyY2VzIjpbIk1hcFNjcmVlbnIudHMiXSwibmFtZXMiOltdLCJtYXBwaW5ncyI6Ijs7SUFJQTs7T0FFRztJQUNIO1FBd0RJOzs7OztXQUtHO1FBQ0gsb0JBQW1CLFFBQTZCO1lBWGhEOztlQUVHO1lBQ0ksY0FBUyxHQUFlLEVBQUUsQ0FBQztZQVM5QixFQUFFLENBQUMsQ0FBQyxPQUFPLFFBQVEsS0FBSyxXQUFXLENBQUMsQ0FBQyxDQUFDO2dCQUNsQyxNQUFNLElBQUksS0FBSyxDQUFDLHlDQUF5QyxDQUFDLENBQUM7WUFDL0QsQ0FBQztZQUNELEVBQUUsQ0FBQyxDQUFDLENBQUMsUUFBUSxDQUFDLEtBQUssQ0FBQyxDQUFDLENBQUM7Z0JBQ2xCLE1BQU0sSUFBSSxLQUFLLENBQUMsK0JBQStCLENBQUMsQ0FBQztZQUNyRCxDQUFDO1lBQ0QsRUFBRSxDQUFDLENBQUMsQ0FBQyxRQUFRLENBQUMsTUFBTSxDQUFDLENBQUMsQ0FBQztnQkFDbkIsTUFBTSxJQUFJLEtBQUssQ0FBQyxnQ0FBZ0MsQ0FBQyxDQUFDO1lBQ3RELENBQUM7WUFFRCxFQUFFLENBQUMsQ0FBQyxRQUFRLENBQUMsU0FBUyxDQUFDLENBQUMsQ0FBQztnQkFDckIsR0FBRyxDQUFDLENBQUMsSUFBTSxNQUFJLElBQUksUUFBUSxDQUFDLFNBQVMsQ0FBQyxDQUFDLENBQUM7b0JBQ3BDLEVBQUUsQ0FBQyxDQUFDLFFBQVEsQ0FBQyxTQUFTLENBQUMsY0FBYyxDQUFDLE1BQUksQ0FBQyxDQUFDLENBQUMsQ0FBQzt3QkFDMUMsSUFBSSxDQUFDLFNBQVMsQ0FBQyxNQUFJLENBQUMsR0FBRyxRQUFRLENBQUMsU0FBUyxDQUFDLE1BQUksQ0FBQyxDQUFDO29CQUNwRCxDQUFDO2dCQUNMLENBQUM7WUFDTCxDQUFDO1lBRUQsSUFBSSxDQUFDLE1BQU0sR0FBRyxRQUFRLENBQUMsTUFBTSxDQUFDO1lBQzlCLElBQUksQ0FBQyxLQUFLLEdBQUcsUUFBUSxDQUFDLEtBQUssQ0FBQztZQUM1QixJQUFJLENBQUMsaUJBQWlCLEdBQUcsUUFBUSxDQUFDLGlCQUFpQixJQUFJLEVBQUUsQ0FBQztZQUMxRCxJQUFJLENBQUMsWUFBWSxHQUFHLFFBQVEsQ0FBQyxZQUFZLElBQUksRUFBRSxDQUFDO1FBQ3BELENBQUM7UUFFRDs7O1dBR0c7UUFDSSxnQ0FBVyxHQUFsQjtZQUNJLElBQUksQ0FBQyxJQUFJLEdBQUcsQ0FBQyxDQUFDO1lBQ2QsSUFBSSxDQUFDLEdBQUcsR0FBRyxDQUFDLENBQUM7WUFDYixJQUFJLENBQUMsS0FBSyxHQUFHLElBQUksQ0FBQyxLQUFLLENBQUM7WUFDeEIsSUFBSSxDQUFDLE1BQU0sR0FBRyxJQUFJLENBQUMsTUFBTSxDQUFDO1lBRTFCLElBQUksQ0FBQyxVQUFVLEVBQUUsQ0FBQztZQUNsQixJQUFJLENBQUMsVUFBVSxFQUFFLENBQUM7WUFFbEIsSUFBSSxDQUFDLFlBQVksRUFBRSxDQUFDO1FBQ3hCLENBQUM7UUFFRDs7V0FFRztRQUNJLCtCQUFVLEdBQWpCO1lBQ0ksSUFBSSxDQUFDLE9BQU8sR0FBRyxDQUFDLElBQUksQ0FBQyxJQUFJLEdBQUcsSUFBSSxDQUFDLEtBQUssQ0FBQyxHQUFHLENBQUMsQ0FBQztRQUNoRCxDQUFDO1FBRUQ7O1dBRUc7UUFDSSwrQkFBVSxHQUFqQjtZQUNJLElBQUksQ0FBQyxPQUFPLEdBQUcsQ0FBQyxJQUFJLENBQUMsR0FBRyxHQUFHLElBQUksQ0FBQyxNQUFNLENBQUMsR0FBRyxDQUFDLENBQUM7UUFDaEQsQ0FBQztRQUVEOztXQUVHO1FBQ0ksaUNBQVksR0FBbkI7WUFDSSxHQUFHLENBQUMsQ0FBQyxJQUFNLENBQUMsSUFBSSxJQUFJLENBQUMsaUJBQWlCLENBQUMsQ0FBQyxDQUFDO2dCQUNyQyxFQUFFLENBQUMsQ0FBQyxJQUFJLENBQUMsaUJBQWlCLENBQUMsY0FBYyxDQUFDLENBQUMsQ0FBQyxDQUFDLENBQUMsQ0FBQztvQkFDM0MsSUFBSSxDQUFDLFdBQVcsQ0FBQyxDQUFDLENBQUMsQ0FBQztnQkFDeEIsQ0FBQztZQUNMLENBQUM7UUFDTCxDQUFDO1FBRUQ7Ozs7OztXQU1HO1FBQ0ksZ0NBQVcsR0FBbEIsVUFBbUIsSUFBWSxFQUFFLEtBQVc7WUFDeEMsSUFBSSxDQUFDLFNBQVMsQ0FBQyxJQUFJLENBQUMsR0FBRyxTQUFTLENBQUMsTUFBTSxLQUFLLENBQUM7a0JBQ3ZDLElBQUksQ0FBQyxpQkFBaUIsQ0FBQyxJQUFJLENBQUMsQ0FBQyxLQUFLLENBQUMsSUFBSSxFQUFFLElBQUksQ0FBQyxZQUFZLENBQUM7a0JBQzNELEtBQUssQ0FBQztRQUNoQixDQUFDO1FBRUQ7Ozs7O1dBS0c7UUFDSSwwQkFBSyxHQUFaLFVBQWEsRUFBVSxFQUFFLEVBQVU7WUFDL0IsRUFBRSxDQUFDLENBQUMsRUFBRSxDQUFDLENBQUMsQ0FBQztnQkFDTCxJQUFJLENBQUMsTUFBTSxDQUFDLEVBQUUsQ0FBQyxDQUFDO1lBQ3BCLENBQUM7WUFFRCxFQUFFLENBQUMsQ0FBQyxFQUFFLENBQUMsQ0FBQyxDQUFDO2dCQUNMLElBQUksQ0FBQyxNQUFNLENBQUMsRUFBRSxDQUFDLENBQUM7WUFDcEIsQ0FBQztRQUNMLENBQUM7UUFFRDs7OztXQUlHO1FBQ0ksMkJBQU0sR0FBYixVQUFjLEVBQVU7WUFDcEIsSUFBSSxDQUFDLElBQUksSUFBSSxFQUFFLENBQUM7WUFDaEIsSUFBSSxDQUFDLEtBQUssSUFBSSxFQUFFLENBQUM7UUFDckIsQ0FBQztRQUVEOzs7O1dBSUc7UUFDSSwyQkFBTSxHQUFiLFVBQWMsRUFBVTtZQUNwQixJQUFJLENBQUMsR0FBRyxJQUFJLEVBQUUsQ0FBQztZQUNmLElBQUksQ0FBQyxNQUFNLElBQUksRUFBRSxDQUFDO1FBQ3RCLENBQUM7UUFDTCxpQkFBQztJQUFELENBaExBLEFBZ0xDLElBQUE7SUFoTFksa0JBQVUsYUFnTHRCLENBQUEiLCJmaWxlIjoiTWFwU2NyZWVuci5qcyIsInNvdXJjZXNDb250ZW50IjpbImltcG9ydCB7XHJcbiAgICBJTWFwU2NyZWVuciwgSU1hcFNjcmVlbnJTZXR0aW5ncywgSVZhcmlhYmxlRnVuY3Rpb25zLCBJVmFyaWFibGVzXHJcbn0gZnJvbSBcIi4vSU1hcFNjcmVlbnJcIjtcclxuXHJcbi8qKlxyXG4gKiBBIGZsZXhpYmxlIGNvbnRhaW5lciBmb3IgbWFwIGF0dHJpYnV0ZXMgYW5kIHZpZXdwb3J0LlxyXG4gKi9cclxuZXhwb3J0IGNsYXNzIE1hcFNjcmVlbnIgaW1wbGVtZW50cyBJTWFwU2NyZWVuciB7XHJcbiAgICAvKipcclxuICAgICAqIEEgbGlzdGluZyBvZiB2YXJpYWJsZSBGdW5jdGlvbnMgdG8gYmUgY2FsY3VsYXRlZCBvbiBzY3JlZW4gcmVzZXRzLlxyXG4gICAgICovXHJcbiAgICBwdWJsaWMgdmFyaWFibGVGdW5jdGlvbnM6IElWYXJpYWJsZUZ1bmN0aW9ucztcclxuXHJcbiAgICAvKipcclxuICAgICAqIEFyZ3VtZW50cyB0byBiZSBwYXNzZWQgaW50byB2YXJpYWJsZSBjb21wdXRhdGlvbiBGdW5jdGlvbnMuXHJcbiAgICAgKi9cclxuICAgIHB1YmxpYyB2YXJpYWJsZUFyZ3M6IGFueVtdO1xyXG5cclxuICAgIC8qKlxyXG4gICAgICogVG9wIGJvcmRlciBtZWFzdXJlbWVudCBvZiB0aGUgYm91bmRpbmcgYm94LlxyXG4gICAgICovXHJcbiAgICBwdWJsaWMgdG9wOiBudW1iZXI7XHJcblxyXG4gICAgLyoqXHJcbiAgICAgKiBSaWdodCBib3JkZXIgbWVhc3VyZW1lbnQgb2YgdGhlIGJvdW5kaW5nIGJveC5cclxuICAgICAqL1xyXG4gICAgcHVibGljIHJpZ2h0OiBudW1iZXI7XHJcblxyXG4gICAgLyoqXHJcbiAgICAgKiBCb3R0b20gYm9yZGVyIG1lYXN1cmVtZW50IG9mIHRoZSBib3VuZGluZyBib3guXHJcbiAgICAgKi9cclxuICAgIHB1YmxpYyBib3R0b206IG51bWJlcjtcclxuXHJcbiAgICAvKipcclxuICAgICAqIExlZnQgYm9yZGVyIG1lYXN1cmVtZW50IG9mIHRoZSBib3VuZGluZyBib3guXHJcbiAgICAgKi9cclxuICAgIHB1YmxpYyBsZWZ0OiBudW1iZXI7XHJcblxyXG4gICAgLyoqXHJcbiAgICAgKiBDb25zdGFudCBob3Jpem9udGFsIG1pZHBvaW50IG9mIHRoZSBib3VuZGluZyBib3gsIGVxdWFsIHRvIChsZWZ0ICsgcmlnaHQpIC8gMi5cclxuICAgICAqL1xyXG4gICAgcHVibGljIG1pZGRsZVg6IG51bWJlcjtcclxuXHJcbiAgICAvKipcclxuICAgICAqIENvbnN0YW50IHZlcnRpY2FsIG1pZHBvaW50IG9mIHRoZSBib3VuZGluZyBib3gsIGVxdWFsIHRvICh0b3AgKyBib3R0b20pIC8gMi5cclxuICAgICAqL1xyXG4gICAgcHVibGljIG1pZGRsZVk6IG51bWJlcjtcclxuXHJcbiAgICAvKipcclxuICAgICAqIENvbnN0YW50IHdpZHRoIG9mIHRoZSBib3VuZGluZyBib3guXHJcbiAgICAgKi9cclxuICAgIHB1YmxpYyB3aWR0aDogbnVtYmVyO1xyXG5cclxuICAgIC8qKlxyXG4gICAgICogQ29uc3RhbnQgaGVpZ2h0IG9mIHRoZSBib3VuZGluZyBib3guXHJcbiAgICAgKi9cclxuICAgIHB1YmxpYyBoZWlnaHQ6IG51bWJlcjtcclxuXHJcbiAgICAvKipcclxuICAgICAqIEFzc29ydGVkIGtub3duIHZhcmlhYmxlcywga2V5ZWQgYnkgbmFtZS5cclxuICAgICAqL1xyXG4gICAgcHVibGljIHZhcmlhYmxlczogSVZhcmlhYmxlcyA9IHt9O1xyXG5cclxuICAgIC8qKlxyXG4gICAgICogUmVzZXRzIHRoZSBNYXBTY3JlZW5yLiBBbGwgbWVtYmVycyBvZiB0aGUgc2V0dGluZ3MgYXJndW1lbnQgYXJlIGNvcGllZFxyXG4gICAgICogdG8gdGhlIE1hcFNjcmVlbnIgaXRzZWxmLCB0aG91Z2ggb25seSB3aWR0aCBhbmQgaGVpZ2h0IGFyZSByZXF1aXJlZC5cclxuICAgICAqIFxyXG4gICAgICogQHBhcmFtIHtJTWFwU2NyZWVuclNldHRpbmdzfSBzZXR0aW5nc1xyXG4gICAgICovXHJcbiAgICBwdWJsaWMgY29uc3RydWN0b3Ioc2V0dGluZ3M6IElNYXBTY3JlZW5yU2V0dGluZ3MpIHtcclxuICAgICAgICBpZiAodHlwZW9mIHNldHRpbmdzID09PSBcInVuZGVmaW5lZFwiKSB7XHJcbiAgICAgICAgICAgIHRocm93IG5ldyBFcnJvcihcIk5vIHNldHRpbmdzIG9iamVjdCBnaXZlbiB0byBNYXBTY3JlZW5yLlwiKTtcclxuICAgICAgICB9XHJcbiAgICAgICAgaWYgKCFzZXR0aW5ncy53aWR0aCkge1xyXG4gICAgICAgICAgICB0aHJvdyBuZXcgRXJyb3IoXCJObyB3aWR0aCBnaXZlbiB0byBNYXBTY3JlZW5yLlwiKTtcclxuICAgICAgICB9XHJcbiAgICAgICAgaWYgKCFzZXR0aW5ncy5oZWlnaHQpIHtcclxuICAgICAgICAgICAgdGhyb3cgbmV3IEVycm9yKFwiTm8gaGVpZ2h0IGdpdmVuIHRvIE1hcFNjcmVlbnIuXCIpO1xyXG4gICAgICAgIH1cclxuXHJcbiAgICAgICAgaWYgKHNldHRpbmdzLnZhcmlhYmxlcykge1xyXG4gICAgICAgICAgICBmb3IgKGNvbnN0IG5hbWUgaW4gc2V0dGluZ3MudmFyaWFibGVzKSB7XHJcbiAgICAgICAgICAgICAgICBpZiAoc2V0dGluZ3MudmFyaWFibGVzLmhhc093blByb3BlcnR5KG5hbWUpKSB7XHJcbiAgICAgICAgICAgICAgICAgICAgdGhpcy52YXJpYWJsZXNbbmFtZV0gPSBzZXR0aW5ncy52YXJpYWJsZXNbbmFtZV07XHJcbiAgICAgICAgICAgICAgICB9XHJcbiAgICAgICAgICAgIH1cclxuICAgICAgICB9XHJcblxyXG4gICAgICAgIHRoaXMuaGVpZ2h0ID0gc2V0dGluZ3MuaGVpZ2h0O1xyXG4gICAgICAgIHRoaXMud2lkdGggPSBzZXR0aW5ncy53aWR0aDtcclxuICAgICAgICB0aGlzLnZhcmlhYmxlRnVuY3Rpb25zID0gc2V0dGluZ3MudmFyaWFibGVGdW5jdGlvbnMgfHwge307XHJcbiAgICAgICAgdGhpcy52YXJpYWJsZUFyZ3MgPSBzZXR0aW5ncy52YXJpYWJsZUFyZ3MgfHwgW107XHJcbiAgICB9XHJcblxyXG4gICAgLyoqXHJcbiAgICAgKiBDb21wbGV0ZWx5IGNsZWFycyB0aGUgTWFwU2NyZWVuciBmb3IgdXNlIGluIGEgbmV3IEFyZWEuIFBvc2l0aW9uaW5nIGlzXHJcbiAgICAgKiByZXNldCB0byAoMCwwKSBhbmQgdXNlci1jb25maWd1cmVkIHZhcmlhYmxlcyBhcmUgcmVjYWxjdWxhdGVkLlxyXG4gICAgICovXHJcbiAgICBwdWJsaWMgY2xlYXJTY3JlZW4oKTogdm9pZCB7XHJcbiAgICAgICAgdGhpcy5sZWZ0ID0gMDtcclxuICAgICAgICB0aGlzLnRvcCA9IDA7XHJcbiAgICAgICAgdGhpcy5yaWdodCA9IHRoaXMud2lkdGg7XHJcbiAgICAgICAgdGhpcy5ib3R0b20gPSB0aGlzLmhlaWdodDtcclxuXHJcbiAgICAgICAgdGhpcy5zZXRNaWRkbGVYKCk7XHJcbiAgICAgICAgdGhpcy5zZXRNaWRkbGVZKCk7XHJcblxyXG4gICAgICAgIHRoaXMuc2V0VmFyaWFibGVzKCk7XHJcbiAgICB9XHJcblxyXG4gICAgLyoqXHJcbiAgICAgKiBDb21wdXRlcyBtaWRkbGVYIGFzIHRoZSBtaWRwb2ludCBiZXR3ZWVuIGxlZnQgYW5kIHJpZ2h0LlxyXG4gICAgICovXHJcbiAgICBwdWJsaWMgc2V0TWlkZGxlWCgpOiB2b2lkIHtcclxuICAgICAgICB0aGlzLm1pZGRsZVggPSAodGhpcy5sZWZ0ICsgdGhpcy5yaWdodCkgLyAyO1xyXG4gICAgfVxyXG5cclxuICAgIC8qKlxyXG4gICAgICogQ29tcHV0ZXMgbWlkZGxlWSBhcyB0aGUgbWlkcG9pbnQgYmV0d2VlbiB0b3AgYW5kIGJvdHRvbS5cclxuICAgICAqL1xyXG4gICAgcHVibGljIHNldE1pZGRsZVkoKTogdm9pZCB7XHJcbiAgICAgICAgdGhpcy5taWRkbGVZID0gKHRoaXMudG9wICsgdGhpcy5ib3R0b20pIC8gMjtcclxuICAgIH1cclxuXHJcbiAgICAvKipcclxuICAgICAqIFJlY2FsY3VsYXRlcyBhbGwgdmFyaWFibGVzIGJ5IHBhc3NpbmcgdmFyaWFibGVBcmdzIHRvIHRoZWlyIEZ1bmN0aW9ucy5cclxuICAgICAqL1xyXG4gICAgcHVibGljIHNldFZhcmlhYmxlcygpOiB2b2lkIHtcclxuICAgICAgICBmb3IgKGNvbnN0IGkgaW4gdGhpcy52YXJpYWJsZUZ1bmN0aW9ucykge1xyXG4gICAgICAgICAgICBpZiAodGhpcy52YXJpYWJsZUZ1bmN0aW9ucy5oYXNPd25Qcm9wZXJ0eShpKSkge1xyXG4gICAgICAgICAgICAgICAgdGhpcy5zZXRWYXJpYWJsZShpKTtcclxuICAgICAgICAgICAgfVxyXG4gICAgICAgIH1cclxuICAgIH1cclxuXHJcbiAgICAvKipcclxuICAgICAqIFJlY2FsY3VsYXRlcyBhIHZhcmlhYmxlIGJ5IHBhc3NpbmcgdmFyaWFibGVBcmdzIHRvIGl0cyBGdW5jdGlvbi5cclxuICAgICAqIFxyXG4gICAgICogQHBhcmFtIG5hbWUgICBUaGUgbmFtZSBvZiB0aGUgdmFyaWFibGUgdG8gcmVjYWxjdWxhdGUuXHJcbiAgICAgKiBAcGFyYW0gdmFsdWUgICBBIG5ldyB2YWx1ZSBmb3IgdGhlIHZhcmlhYmxlIGluc3RlYWQgb2YgaXRzIEZ1bmN0aW9uJ3MgcmVzdWx0LlxyXG4gICAgICogQHJldHVybnMgVGhlIG5ldyB2YWx1ZSBvZiB0aGUgdmFyaWFibGUuXHJcbiAgICAgKi9cclxuICAgIHB1YmxpYyBzZXRWYXJpYWJsZShuYW1lOiBzdHJpbmcsIHZhbHVlPzogYW55KTogYW55IHtcclxuICAgICAgICB0aGlzLnZhcmlhYmxlc1tuYW1lXSA9IGFyZ3VtZW50cy5sZW5ndGggPT09IDFcclxuICAgICAgICAgICAgPyB0aGlzLnZhcmlhYmxlRnVuY3Rpb25zW25hbWVdLmFwcGx5KHRoaXMsIHRoaXMudmFyaWFibGVBcmdzKVxyXG4gICAgICAgICAgICA6IHZhbHVlO1xyXG4gICAgfVxyXG5cclxuICAgIC8qKlxyXG4gICAgICogU2hpZnRzIHRoZSBNYXBTY3JlZW5yIGhvcml6b250YWxseSBhbmQgdmVydGljYWxseSB2aWEgc2hpZnRYIGFuZCBzaGlmdFkuXHJcbiAgICAgKiBcclxuICAgICAqIEBwYXJhbSBkeCAgIEhvdyBmYXIgdG8gc2Nyb2xsIGhvcml6b250YWxseS5cclxuICAgICAqIEBwYXJhbSBkeSAgIEhvdyBmYXIgdG8gc2Nyb2xsIHZlcnRpY2FsbHkuXHJcbiAgICAgKi9cclxuICAgIHB1YmxpYyBzaGlmdChkeDogbnVtYmVyLCBkeTogbnVtYmVyKTogdm9pZCB7XHJcbiAgICAgICAgaWYgKGR4KSB7XHJcbiAgICAgICAgICAgIHRoaXMuc2hpZnRYKGR4KTtcclxuICAgICAgICB9XHJcblxyXG4gICAgICAgIGlmIChkeSkge1xyXG4gICAgICAgICAgICB0aGlzLnNoaWZ0WShkeSk7XHJcbiAgICAgICAgfVxyXG4gICAgfVxyXG5cclxuICAgIC8qKlxyXG4gICAgICogU2hpZnRzIHRoZSBNYXBTY3JlZW5yIGhvcml6b250YWxseSBieSBjaGFuZ2luZyBsZWZ0IGFuZCByaWdodCBieSB0aGUgZHguXHJcbiAgICAgKiBcclxuICAgICAqIEBwYXJhbSBkeCAgIEhvdyBmYXIgdG8gc2Nyb2xsIGhvcml6b250YWxseS5cclxuICAgICAqL1xyXG4gICAgcHVibGljIHNoaWZ0WChkeDogbnVtYmVyKTogdm9pZCB7XHJcbiAgICAgICAgdGhpcy5sZWZ0ICs9IGR4O1xyXG4gICAgICAgIHRoaXMucmlnaHQgKz0gZHg7XHJcbiAgICB9XHJcblxyXG4gICAgLyoqXHJcbiAgICAgKiBTaGlmdHMgdGhlIE1hcFNjcmVlbnIgdmVydGljYWxseSBieSBjaGFuZ2luZyB0b3AgYW5kIGJvdHRvbSBieSB0aGUgZHkuXHJcbiAgICAgKiBcclxuICAgICAqIEBwYXJhbSBkeSAgIEhvdyBmYXIgdG8gc2Nyb2xsIHZlcnRpY2FsbHkuXHJcbiAgICAgKi9cclxuICAgIHB1YmxpYyBzaGlmdFkoZHk6IG51bWJlcik6IHZvaWQge1xyXG4gICAgICAgIHRoaXMudG9wICs9IGR5O1xyXG4gICAgICAgIHRoaXMuYm90dG9tICs9IGR5O1xyXG4gICAgfVxyXG59XHJcbiJdfQ==

src/MapScreenr.ts

@@ -1,4 +0,2 @@
-import {
-    IMapScreenr, IMapScreenrSettings, IVariableFunctions, IVariables
-} from "./IMapScreenr";
+import { IMapScreenr, IMapScreenrSettings, IVariableFunctions, IVariables } from "./IMapScreenr";
 
@@ -12,10 +10,5 @@ /**
      */
-    public variableFunctions: IVariableFunctions;
+    public readonly variableFunctions: IVariableFunctions;
 
     /**
-     * Arguments to be passed into variable computation Functions.
-     */
-    public variableArgs: any[];
-
-    /**
      * Top border measurement of the bounding box.
@@ -53,3 +46,3 @@ */
      */
-    public width: number;
+    public readonly width: number;
 
@@ -59,3 +52,3 @@ /**
      */
-    public height: number;
+    public readonly height: number;
 
@@ -65,24 +58,13 @@ /**
      */
-    public variables: IVariables = {};
+    public readonly variables: IVariables = {};
 
     /**
-     * Resets the MapScreenr. All members of the settings argument are copied
-     * to the MapScreenr itself, though only width and height are required.
-     * 
-     * @param {IMapScreenrSettings} settings
+     * Initializes a new instance of the MapScreenr class.
+     *
+     * @param settings   Settings to be used for initialization.
      */
     public constructor(settings: IMapScreenrSettings) {
-        if (typeof settings === "undefined") {
-            throw new Error("No settings object given to MapScreenr.");
-        }
-        if (!settings.width) {
-            throw new Error("No width given to MapScreenr.");
-        }
-        if (!settings.height) {
-            throw new Error("No height given to MapScreenr.");
-        }
-
-        if (settings.variables) {
+        if (settings.variables !== undefined) {
             for (const name in settings.variables) {
-                if (settings.variables.hasOwnProperty(name)) {
+                if ({}.hasOwnProperty.call(settings.variables, name)) {
                     this.variables[name] = settings.variables[name];
@@ -95,4 +77,5 @@ }
         this.width = settings.width;
-        this.variableFunctions = settings.variableFunctions || {};
-        this.variableArgs = settings.variableArgs || [];
+        this.variableFunctions = settings.variableFunctions === undefined
+            ? {}
+            : settings.variableFunctions;
     }
@@ -131,9 +114,7 @@
     /**
-     * Recalculates all variables by passing variableArgs to their Functions.
+     * Recalculates all variables.
      */
     public setVariables(): void {
         for (const i in this.variableFunctions) {
-            if (this.variableFunctions.hasOwnProperty(i)) {
-                this.setVariable(i);
-            }
+            this.setVariable(i);
         }
@@ -144,3 +125,3 @@ }
      * Recalculates a variable by passing variableArgs to its Function.
-     * 
+     *
      * @param name   The name of the variable to recalculate.
@@ -152,3 +133,3 @@ * @param value   A new value for the variable instead of its Function's result.
         this.variables[name] = arguments.length === 1
-            ? this.variableFunctions[name].apply(this, this.variableArgs)
+            ? this.variableFunctions[name]()
             : value;
@@ -159,3 +140,3 @@ }
      * Shifts the MapScreenr horizontally and vertically via shiftX and shiftY.
-     * 
+     *
      * @param dx   How far to scroll horizontally.
@@ -165,7 +146,7 @@ * @param dy   How far to scroll vertically.
     public shift(dx: number, dy: number): void {
-        if (dx) {
+        if (dx !== 0) {
             this.shiftX(dx);
         }
 
-        if (dy) {
+        if (dy !== 0) {
             this.shiftY(dy);
@@ -177,3 +158,3 @@ }
      * Shifts the MapScreenr horizontally by changing left and right by the dx.
-     * 
+     *
      * @param dx   How far to scroll horizontally.
@@ -188,3 +169,3 @@ */
      * Shifts the MapScreenr vertically by changing top and bottom by the dy.
-     * 
+     *
      * @param dy   How far to scroll vertically.
@@ -191,0 +172,0 @@ */

tsconfig.json

 {
     "compilerOptions": {
         "declaration": true,
+        "experimentalDecorators": true,
+        "jsx": "react",
+        "lib": ["dom", "es2015.collection", "es2015.promise", "es5"],
         "module": "amd",
@@ -10,10 +13,14 @@ "moduleResolution": "node",
         "noFallthroughCasesInSwitch": true,
-        "noUnusedLocals": true,
-        "noUnusedParameters": true,
+        "pretty": true,
         "strictNullChecks": true,
-        "target": "es3"
+        "target": "es5"
     },
+    "exclude": [
+        "dist",
+        "node_modules"
+    ],
     "include": [
-        "./src/**/*.ts"
+        "./src/**/*.ts",
+        "./src/**/*.tsx"
     ]
 }

tslint.json

 {
+    "extends": "./node_modules/shenanigans-manager/setup/tslint.json",
     "rules": {
-        "align": [
-            true,
-            "parameters",
-            "arguments",
-            "statements"
-        ],
-        "class-name": true,
-        "comment-format": [true, "check-space"],
-        "curly": true,
-        "eofline": true,
-        "forin": true,
-        "indent": [true, 4],
-        "interface-name": [true, "always-prefix"],
-        "jsdoc-format": true,
-        "label-position": true,
-        "max-line-length": [true, 140],
-        "member-access": true,
-        "member-ordering": [
-            true,
-            "public-before-private",
-            "static-before-instance",
-            "variables-before-functions"
-        ],
-        "new-parens": true,
-        "no-angle-bracket-type-assertion": true,
-        "no-arg": true,
-        "no-conditional-assignment": true,
-        "no-consecutive-blank-lines": true,
-        "no-console": [
-            true,
-            "debug",
-            "info",
-            "time",
-            "timeEnd",
-            "trace"
-        ],
-        "no-construct": true,
-        "no-debugger": true,
-        "no-duplicate-variable": true,
-        "no-empty": true,
-        "no-eval": true,
-        "no-internal-module": true,
-        "no-null-keyword": true,
-        "no-shadowed-variable": true,
-        "no-string-literal": true,
-        "no-switch-case-fall-through": true,
-        "no-trailing-whitespace": true,
-        "no-unsafe-finally": true,
-        "no-unused-expression": true,
-        "no-unused-new": true,
-        "no-use-before-declare": true,
-        "no-var-keyword": true,
-        "one-line": [
-            true,
-            "check-catch",
-            "check-else",
-            "check-finally",
-            "check-open-brace",
-            "check-whitespace"
-        ],
-        "one-variable-per-declaration": true,
-        "ordered-imports": true,
-        "quotemark": [true, "double"],
-        "semicolon": [true, "always"],
-        "switch-default": true,
-        "trailing-comma": true,
-        "triple-equals": [true, "allow-null-check"],
-        "typedef": [
-            true,
-            "call-signature",
-            "parameter",
-            "property-declaration",
-            "variable-declaration",
-            "member-variable-declaration"
-        ],
-        "typedef-whitespace": [
-            true,
-            {
-                "call-signature": "nospace",
-                "index-signature": "nospace",
-                "parameter": "nospace",
-                "property-declaration": "nospace",
-                "variable-declaration": "nospace"
-            }
-        ],
-        "use-isnan": true,
-        "whitespace": [
-            true,
-            "check-branch",
-            "check-decl",
-            "check-operator",
-            "check-separator",
-            "check-type"
-        ]
+        "no-any": false
     }
-}
+}

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

Major refactor

Supply chain risk

Package has recently undergone a major refactor. It may be unstable or indicate significant internal changes. Use caution when updating to versions that include significant changes.

Found 1 instance in 1 package

Minified code

Quality

This package contains minified code. This may be harmless in some cases where minified code is included in packaged libraries, however packages on npm should not minify code.

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

Minified code

Quality

This package contains minified code. This may be harmless in some cases where minified code is included in packaged libraries, however packages on npm should not minify code.

Found 1 instance in 1 package

Worsened metrics

Total package byte prevSize

475601

decreased by-7.33%

Dev dependency count

19

increased by1800%

Number of package files

32

decreased by-33.33%

Lines of code

2079

decreased by-20.74%

Number of lines in readme file

16

decreased by-27.27%

Number of medium supply chain risk alerts

4

increased by33.33%

No dependency changes