Repository navigation
Make Lib.nestedProperty an npm module of its own #2138
Description
So that users and plotly.js consumer projects can use it!
The tricky part will be to factor out the core nested property logic from the plotly.js-specific parts (namely plot_api/container_array_match.js).
Wrapping nestedProperty in a constructor should do the trick. For example, plotly.js/src/lib/nested_property.js could become:
var NestedProperty = require('@plotly/nested-property');
var containerArrayMatch = require('../plot_api/container_array_match');
module.exports = new NestedProperty({containerArrayMatch: containerArrayMatch});
Activity
alexcjohnson commentedon Nov 1, 2017
@etpinard sounds great! The only change I'd suggest is instead of
containerArrayMatch, factor outisDeletable- there's more plotly.js-specific logic in there than justcontainerArrayMatch. That said, this logic is fairly obtuse and always makes me wonder how much would really be lost if we ditched it or at least simplified it...etpinard commentedon May 8, 2019
Probably not going to happen in the near future. Closing.