Skip to content

Add list of sounds you heard at the end#2

Open
AndaxH wants to merge 1 commit into
masterfrom
cards-you-saw
Open

Add list of sounds you heard at the end#2
AndaxH wants to merge 1 commit into
masterfrom
cards-you-saw

Conversation

@AndaxH
Copy link
Copy Markdown
Collaborator

@AndaxH AndaxH commented Dec 10, 2017

No description provided.

@AndaxH AndaxH requested a review from rissole December 10, 2017 11:33
@rissole
Copy link
Copy Markdown
Collaborator

rissole commented Dec 11, 2017

I'm still not quite happy with the look of this and I think we need a little play button next to the sounds to play them again.

Also I think we should only show the ones you got wrong at the end. It makes the UI much cleaner and players already know the ones they got correct, so there isn't much/any value in showing every sound.

Copy link
Copy Markdown
Collaborator

@rissole rissole left a comment

Choose a reason for hiding this comment

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

fuck 2017 rissole, im never happy with anything anyway. Approved

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