Looks good, that is kind of consistent with the general behaviour of flagging the PGN mistake by flashing the A8 square.Jeremy Bernstein wrote:OK, both of those problems are "fixed" (the second one really, the first one is just kind of fixed (throws errors because the PGN is bad, but loads the board and lets you begin to debug your game). I'll look more closely, time permitting, at a better solution for invalid PGN data.
I had a couple of people asking for the variation feature, hope you dont mind I pointed them to get a copy of your pgn4web.js file and see if that works for them.
Havent got the time (and patience) to make your code general enough and configurable (things like variations colors/fonts and so on) for the main pgn4web version. Other things need to be done also, such as extending the SetInitialHalfmove function to allow pointing to a move into a variation and so on.
One final comment, you display variations in square brackets... the PGN standard uses round brackets for that, people used to PGN files would probably find it easier to read with square brackets, but in the end it's just cosmetic.