small code cleanup, simplify constructor and nested statements

shaack / cm-chessboard

A JavaScript chessboard without dependencies. Rendered in SVG, coded in ES6. Views FEN, handles move input, animated, responsive, expandable

MIT License

209 stars 63 forks source link

Closed Disservin closed 10 months ago

Disservin commented 10 months ago

remove console.logs
explicitly cast to string
define methods outside of constructor, I dont see why this was required other than perhaps calling view instead of this.view
use early returns for deeply nested if statements

shaack commented 10 months ago

Looks good on first sight. 👍 Will do some deeper testing and then merge it, if everything works.