Updating the position of only one piece outside `onDrop` doesn't update the UI about chessboardjsx HOT 11 CLOSED

Ok, thanks

I'm looking into it now.

from chessboardjsx.

going to have to figure this out later today. it's a bug with how the board his handling the prevState and currentState

from chessboardjsx.

I'm going to expose a new prop called undo to fix this for now.

I wrote the chessboard to compare positions only if the current position is different than the previous position and when the current position becomes the previous position then there is no comparing.

I'll try to code up a demo sometime soon

from chessboardjsx.

Check out this branch : https://github.com/willb335/chessboardjsx/tree/undo
Run npm run start
Click the Undo demo

In the demo I set all pieces except the knights to 'undo' by specifying an undo prop

Wish I could have integrated a fix without an extra prop but couldn't see an easy way without a substantial rewrite.

Let me know if this works!

from chessboardjsx.

Check out the fork of your demo below. I believe it was an async issue with setState. I changed

this.setState(...)
setTimeout(this.setState(...), 1000)

to

this.setState(..., () => setTimeout(this.setState(...), 1000) )

just put the setTimeout into the initial setState's callback.

let me know if this solves the issue
https://codesandbox.io/s/2xpxvq06wj

from chessboardjsx.

Hi!

Thank you for checking this issue.

On my end I'm still seeing the same problem. Opening the Dev Console, I see that it prints the fen correctly, but the UI stills shows the older state.

Here for instance I moved Kg2, and I'm hovering the f2 pawn:

Thank you very much!

from chessboardjsx.

*prevPos and currentPos

from chessboardjsx.

Thank you very much! Will try and get back to you.

from chessboardjsx.

Just tested and this works. Thank you for working on this fix!

from chessboardjsx.

great! i'll update the docs and push up a new release soon

from chessboardjsx.

closed by:
2fff88a

from chessboardjsx.