@jrkasprzyk/parcoord-es 3.0.0 → 3.0.2

package/README.md

@@ -1,6 +1,16 @@
-# parcoords-es
+# @jrkasprzyk/parcoord-es
 
-ES6 module of Syntagmatic's [Parallel Coordinates](https://github.com/syntagmatic/parallel-coordinates) (aka. parcoords). This library is completely based on D3 V5 API. 
+[![npm version](https://img.shields.io/npm/v/%40jrkasprzyk%2Fparcoord-es.svg)](https://www.npmjs.com/package/@jrkasprzyk/parcoord-es)
+
+A maintained fork of [BigFatDog/parcoords-es](https://github.com/BigFatDog/parcoords-es), an ES6 module of Syntagmatic's [Parallel Coordinates](https://github.com/syntagmatic/parallel-coordinates) (aka. parcoords). This library is completely based on the D3 V7 API.
+
+## About this fork
+
+The upstream `parcoord-es` package is built on D3 V5 and is no longer actively maintained. This fork:
+
+* Migrates the library to **D3 V7** modules
+* Updates the build toolchain (Rollup 4, Babel 7, ESLint 8, Mocha 11) and resolves all known dependency vulnerabilities
+* Is published to npm as [`@jrkasprzyk/parcoord-es`](https://www.npmjs.com/package/@jrkasprzyk/parcoord-es)
 
 
 ## Features
@@ -20,13 +30,13 @@ This section only lists api that are <b>deviated</b> from the original parallel
 
 <a name="parcoords_brush_extents" href="#parcoords_brush_extents">#</a> parcoords.<b>brushExtents</b>(extents) supports 1D multi brushes:
 
-1D brush [<>](https://github.com/BigFatDog/parcoords-es/blob/master/demo/setterForBrushes.html "Source")
+1D brush [<>](https://github.com/jrkasprzyk/parcoords-es/blob/master/demo/setterForBrushes.html "Source")
 ```javascript
   .brushMode("1D-axes")
   .brushExtents({"2": [3,4]});
 ```
 
-1D multi brush [<>](https://github.com/BigFatDog/parcoords-es/blob/master/demo/setterForMultiBrushes.html "Source")
+1D multi brush [<>](https://github.com/jrkasprzyk/parcoords-es/blob/master/demo/setterForMultiBrushes.html "Source")
 ```javascript
   .brushMode("1D-axes-multi")
   .brushExtents({"2": [[3,4], [6,8], [10, 14]]});
@@ -56,7 +66,7 @@ const data = [...];
 const pc = ParCoords().data(data)...;
 parcoords.mark(data.filter(d => d.volume > 10));
 ```
-Please refer to [marking demo](https://github.com/BigFatDog/parcoords-es/blob/develop/demo/marking.html "Source") for details
+Please refer to [marking demo](https://github.com/jrkasprzyk/parcoords-es/blob/master/demo/marking.html "Source") for details
 
 <a name="parcoords_unmark" href="#parcoords_unmark">#</a> parcoords.<b>unmark</b>()
 clears all permanently highlighted data that is added by <a href="#parcoords_marking">mark([values])</a>
@@ -76,27 +86,27 @@ parcoords.on('brushstart', function(brushed, args){
     } = args;
 })
 ```
-Please refer to [brushing with arguments demo](https://github.com/BigFatDog/parcoords-es/blob/develop/demo/brush-with-arguments.html "Source") for details
+Please refer to [brushing with arguments demo](https://github.com/jrkasprzyk/parcoords-es/blob/master/demo/brush-with-arguments.html "Source") for details
 
 ## Usage
 
 ### ES6
 1. Install library in your project
 ```
-npm install parcoord-es --save
+npm install @jrkasprzyk/parcoord-es --save
 ```
 
 2. import module
 
 ```
-import 'parcoord-es/dist/parcoords.css';
-import ParCoords from 'parcoord-es';
+import '@jrkasprzyk/parcoord-es/dist/parcoords.css';
+import ParCoords from '@jrkasprzyk/parcoord-es';
 
 const chart = ParCoords()....
 ```
 ### Standalone
 
-parcoords.standalone.js contains all dependencies and can be used directly in your html page. Please note that only essential D3 V5 modules are bundled, your global namespace won't be polluted.
+parcoords.standalone.js contains all dependencies and can be used directly in your html page. Please note that only essential D3 V7 modules are bundled, your global namespace won't be polluted.
 ```
 <link rel="stylesheet" type="text/css" href="./parcoords.css">
 <script src="./parcoords.standalone.js"></script>
@@ -104,8 +114,6 @@ parcoords.standalone.js contains all dependencies and can be used directly in yo
 var parcoords = ParCoords()("#example")
 ```
 
-You are free to use either D3 V3 or D3 V5 in your html. demo/superformula.html demonstrates how to use parcoords-es with d3 V3.
-
 ## Development
 
 Follow this instruction to setup development environment for parcoords-es
@@ -142,7 +150,7 @@ npm run test:cover
 
 ## Built With
 
-* [D3 V5](https://d3js.org/) - D3 modules are used
+* [D3 V7](https://d3js.org/) - D3 modules are used
 * [Rollup](https://github.com/rollup/rollup) - Module bundler
 
 
@@ -151,7 +159,7 @@ npm run test:cover
 This project is licensed under the MIT License - see the [LICENSE.md](LICENSE.md) file for details
 
 ## Contributors
-This project gets supports from open-source community. Many thanks to our [contributors](https://github.com/BigFatDog/parcoords-es/graphs/contributors)
+This project gets supports from open-source community. Many thanks to the [upstream contributors](https://github.com/BigFatDog/parcoords-es/graphs/contributors)
 
 ## Contribution Notes
 Run `npm run pretty` before committing.

package/dist/parcoords.esm.js

@@ -265,10 +265,12 @@ var brushFor$1 = function brushFor(state, config, pc, events, brushGroup) {
     }
     var brushRangeMax = config.dimensions[axis].type === 'string' ? config.dimensions[axis].yscale.range()[config.dimensions[axis].yscale.range().length - 1] : config.dimensions[axis].yscale.range()[0];
     var _brush = brushY(_selector).extent([[-15, 0], [15, brushRangeMax]]);
-    var convertBrushArguments = function convertBrushArguments(args) {
-      var args_array = Array.prototype.slice.call(args);
-      var axis = args_array[0];
-      var raw = brushSelection(args_array[2][0]) || [];
+
+    // d3 v6+ invokes listeners as (event, datum); the v5 (datum, index, nodes)
+    // layout this used to slice out of `arguments` no longer exists. The axis name
+    // is already in scope above, and the node is the listener's `this`.
+    var convertBrushArguments = function convertBrushArguments(node) {
+      var raw = brushSelection(node) || [];
 
       // handle hidden axes which will not have a yscale
       var yscale = null;
@@ -279,8 +281,8 @@ var brushFor$1 = function brushFor(state, config, pc, events, brushGroup) {
       // ordinal scales do not have invert
       var scaled = invertByScale(raw, yscale);
       return {
-        axis: args_array[0],
-        node: args_array[2][0],
+        axis: axis,
+        node: node,
         selection: {
           raw: raw,
           scaled: scaled
@@ -288,17 +290,19 @@ var brushFor$1 = function brushFor(state, config, pc, events, brushGroup) {
       };
     };
     _brush.on('start', function (event) {
-      if (event.sourceEvent !== null) {
-        events.call('brushstart', pc, config.brushed, convertBrushArguments(arguments));
+      // d3 v6+ leaves sourceEvent undefined for a programmatic brush.move,
+      // where v5 set it to null; `!=` catches both.
+      if (event.sourceEvent != null) {
+        events.call('brushstart', pc, config.brushed, convertBrushArguments(this));
         if (typeof event.sourceEvent.stopPropagation === 'function') {
           event.sourceEvent.stopPropagation();
         }
       }
     }).on('brush', function () {
-      brushUpdated$1(config, pc, events, convertBrushArguments(arguments))(selected$4(state, config, brushGroup)());
+      brushUpdated$1(config, pc, events, convertBrushArguments(this))(selected$4(state, config, brushGroup)());
     }).on('end', function () {
       brushUpdated$1(config, pc, events)(selected$4(state, config, brushGroup)());
-      events.call('brushend', pc, config.brushed, convertBrushArguments(arguments));
+      events.call('brushend', pc, config.brushed, convertBrushArguments(this));
     });
     state.brushes[axis] = _brush;
     state.brushNodes[axis] = _selector.node();
@@ -715,7 +719,9 @@ var _newBrush = function newBrush(state, config, pc, events, brushGroup) {
       }];
     }
     brush.on('start', function (event) {
-      if (event.sourceEvent !== null) {
+      // d3 v6+ leaves sourceEvent undefined for a programmatic brush.move,
+      // where v5 set it to null; `!=` catches both.
+      if (event.sourceEvent != null) {
         events.call('brushstart', pc, config.brushed);
         if (typeof event.sourceEvent.stopPropagation === 'function') {
           event.sourceEvent.stopPropagation();
@@ -2023,11 +2029,16 @@ var brushable = function brushable(config, pc, flags) {
       if (config.dimensions[d] !== undefined) {
         config.dimensions[d]['brush'] = brushY(select(this)).extent([[-15, 0], [15, config.dimensions[d].yscale.range()[0]]]);
         select(this).call(config.dimensions[d]['brush'].on('start', function (event) {
-          if (event.sourceEvent !== null && !event.sourceEvent.ctrlKey) {
+          // `!=` also catches the undefined d3 v6+ uses for a
+          // programmatic brush.move, which `!== null` let through
+          // straight into the .ctrlKey dereference below.
+          if (event.sourceEvent != null && !event.sourceEvent.ctrlKey) {
             pc.brushReset();
           }
         }).on('brush', function (event) {
-          if (!event.sourceEvent.ctrlKey) {
+          // A programmatic brush.move leaves sourceEvent undefined in
+          // d3 v6+; treat that as "no Ctrl key" rather than dereferencing.
+          if (!(event.sourceEvent && event.sourceEvent.ctrlKey)) {
             pc.brush();
           }
         }).on('end', function (event) {
@@ -2035,7 +2046,7 @@ var brushable = function brushable(config, pc, flags) {
           // store important brush information and
           // the html element of the selection,
           // to make a dummy selection element
-          if (event.sourceEvent.ctrlKey) {
+          if (event.sourceEvent && event.sourceEvent.ctrlKey) {
             var html = select(this).select('.selection').nodes()[0].outerHTML;
             html = html.replace('class="selection"', 'class="selection dummy' + ' selection-' + config.brushes.length + '"');
             var dat = select(this).nodes()[0].__data__;
@@ -3743,7 +3754,7 @@ var scale = function scale(config, pc) {
   };
 };
 
-const version = "3.0.0";
+const version = "3.0.2";
 
 var DefaultConfig = {
   data: [],