onPaste doesn't provide selected cell's row and column index about react-datasheet

[zachguo]

Actually my problem is that, when editing cell content (after doubleclick), one cannot paste anything into the cell.

(I tried pasting react-datasheet into A1 but failed, so basically expected behavior of input box is violated)

[nadbm]

@zachguo
One way to fix this would be to add the indexes right here:
https://github.com/nadbm/react-datasheet/blob/master/src/DataSheet.js#L109

The alternative is to store the index on the cell's data itself since onPaste returns {cellData, pastedData} on for each cell. I guess it can be cumbersome as each cell needs to be initialized with the proper indexes.

Edit: The reason I did not add indexes initially is that sometimes the pasted cells go BEYOND the available cells, so sometimes there is no index that fit the current cells. Feel free to make a PR for this, and null indexes when the paste is out of bounds

[nadbm]

I see, that's a different issue, and definitely a bug to be fixed. Basically should not be calling internal onPaste function if the cell is in editing mode

[zachguo]

Could you point to me where to change to get this fixed? I tried searching input and editing, but it's still not clear to me where to change.

[nadbm]

@zachguo I just pushed a commit, I'll get around to adding tests for it and pushing it to npm. But that should do it

[zachguo]

@nadbm Amazing, thanks!