virtual-dom icon indicating copy to clipboard operation
virtual-dom copied to clipboard

Run time error when trying the example from billstclair/elm-sortable-table.

Open digitalsatori opened this issue 5 years ago • 2 comments

I encountered runtime error when I test the ‘tripl planner’ example in billstclair/elm-sortable-table. The “TypeError: Argument 1 (‘node’) to Node.appendChild must be an instance of Node” occurred, when clicking on ‘Rating’ column after clicking on ‘Price’ column. The gif is attached.

Tested in Safari, google chrome and firefox with extension disable, the same error.

sortable-table

digitalsatori avatar Mar 06 '19 01:03 digitalsatori

I managed to reproduce this locally. I can't figure why this happens yet, but I narrowed it down to Html.Keyed.node. Changing line https://github.com/billstclair/elm-sortable-table/blob/b9c4f87eb55b7f96b9f0f5ec717617ea0f08bad6/src/Table.elm#L466 to use Html.node instead of Keyed.node fixes the issue.

This should probably be addressed elsewhere.

boianr avatar Mar 06 '19 15:03 boianr

@boianr Thank you for taking time to confirm the issue and narrow it down to the problematic code. I'm guessing that the issue may not even belong here. Maybe https://github.com/elm/virtual-dom is a more proper place.

digitalsatori avatar Mar 10 '19 13:03 digitalsatori