Setting NodeList `length` in strict mode throws TypeError about d3-selection HOT 7 CLOSED

I’ll investigate, but a word of caution: as the README says, this module is currently an experimental API and is likely to change before 4.0 is released. This module and d3-transition are likely to be the last of the 4.0 modules to be released (since, in some sense, they are the most important to get the API right). So I wouldn’t recommend using this module in production yet.

I am working on implementing the other modules first, so I can’t be as responsive fixing issues in the unreleased modules such as this one.

That said, it is a bug if the code is trying to set the length of a NodeList, clearly. It should have arrayify’d the arrays first before mutating them.

from d3-selection.

Also, it looks like you’re trying to call selection.text on the enter selection before appending. That shouldn’t fail, but it also won’t have any effect because you’re setting the textContent property of a placeholder node. You’ll want to say something like this instead:

selectAll('p')
    .data(data)
  .enter().append('p')
    .text(d => d)

from d3-selection.

Sure thing, this isn't a production app, so no damage there. The new stuff is too tempting to leave alone though :)

Also, the example was thrown together, that's not app code, the failure is coming in data, before it even hits the enter section but I'll fix the example.

As far as I can tell, the enter and exit calls are arryifying the selection around emptyNode, but the update call is simply this._root, which is a NodeList.

I'm really looking forward to using this stuff in production, happy to help bugbash if that's in the cards.

from d3-selection.

Cool, just making sure. Another gotcha in your test case (probably unrelated to the bug) is that you didn’t select a node to append to, so your entering paragraph elements will get added to the document element rather than the body. The fix:

select('body').selectAll('p')
    .data(data)
  .enter().append('p')
    .text(d => d);

from d3-selection.

Ah. Yeah in the app its a normal selection on a group of bar charts (similar to the crossfilter example). Works right if I leave d3.selectAll in there.

from d3-selection.

The bug is only exhibited when calling selection.data on a top-level (flat) selection, rather than one nested under a parent element. That typically never happens because you instead specify a parent to append to. But it shouldn’t fail. Fix incoming.

from d3-selection.

from d3-selection.