costava / solitaire Goto Github PK

Add license.

When npm run build-dev is ran for the first time, it fails because the prep folder does not exist.
The build succeeds when the command is run a second time.

Building should succeed the first time.

An init script could be made that must be run once after cloning the repo, or the build script could create prep if it does not exist.

When dragging a card and the mouse cursor is over the "Quit" or "New Game" buttons, the card does not move.
Dragging the cards around should be independent of the buttons.
Also, the button appears to get stuck in 'pressed' state until you hover it.

The game would be more fun if it only generated shuffled decks that are possible to win.

The getShuffledDeck method of the Solitaire class is where shuffled decks are generated. Currently, decks are randomly shuffled. It is possible to create an impossible deck.

Any necessary random number generation should come from the existing pseudorandom number generator member srng.

Hi, you have dist folder that should be only on local machine and not in Git

When npm install is ran, multiple of the devDependencies are listed as deprecated.

Update the dependencies.