Skip to content

Update readme#30

Open
Siya-ng wants to merge 36 commits into
wdi-sg:masterfrom
Siya-ng:master
Open

Update readme#30
Siya-ng wants to merge 36 commits into
wdi-sg:masterfrom
Siya-ng:master

Conversation

@Siya-ng
Copy link
Copy Markdown

@Siya-ng Siya-ng commented Sep 29, 2017

No description provided.

Comment thread assets/js/script.js Outdated
function noAvailableMove (){
$gridbox.css({"filter": "grayscale(1)"})
$noMove = $('<div class="noMove">')
$noMoveTextOne = $('<h2 class="noMoveText">No more moves.</h3>')
Copy link
Copy Markdown

@jpung jpung Oct 9, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No description provided.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Forgot to change it. Supposed to be h2. But it still appear... why?

@shumin13
Copy link
Copy Markdown

Glow

  • Interesting game
  • Good implementation of the animations
  • Covered normal, error and edge cases well

Grow & Things to look out for

  • The page will shift when one clicks on the instructions
  • The instructions will appear behind the game board when one clicks on the instructions after the game has started
  • Can look into adding more levels
  • Can look into keeping track of high score
  • May want to add in comments to provide a brief explanation of each function
  • Need to work on the commit messages

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants