paliwalsparsh / dots-and-boxes Goto Github PK

App.css is not being used, it is just a left over file from create-react-app boilerplate. You just need to remove it and make sure things are working fine.

All you need to do is follow the video here by Kent C. Dotts: https://egghead.io/lessons/javascript-writing-conventional-commits-with-commitizen

Currently it is not clear to which player current turn belongs to. This creates confusion. Maybe we can just transform ( increase ) the scale of "Player1" or "Player2" along with score present on top of Game screen. This will user understand to whom the current turn belongs to.

with the app running on one page, having different .css files only forms a race condition. So whichever .css file is imported last wins

Solutions:
condense the CSS into one file
or
scrub the HTML header of imports each time a new "scene" in the game is created and re-import the correct CSS file

Currently player names are hard coded as Player1 and Player2. We can give two inputs on main menu, so users can set names. By default values of these inputs would be Player1 and Player2.

Currently the default color of button is grey. As a result of this, "Start Game" button feels like it is disabled by default with the grey color scheme, and may trick people into thinking they have to do something else before they can start.

Same with the button on Game over screen. We can make the default color of button as orange and onHover and onActive can have darker shade of the same.