braxhawk / Blackjack

0 stars 0 forks source link

Some suggestions and one main issue #16

Open bethany39 opened 4 years ago

bethany39 commented 4 years ago

I made a more detailed eval in the post since I thought you would see it, but I'll try and remember everything I said.

A title for the game would be nice on the home screen. Also, I think you should put the instructions above the name and play button in case they don't "see" it before they click play. Also defining Hit and Stand would be nice in case people don't know and mentioning how much Ace is worth in Blackjack as well.

The main issue is the cards aren't recognized by VO; it completely skips over them, so the user doesn't know what they have.

I would suggest putting a media query so it looks better on mobile view. VO is able to get to everything with swipe gestures, but if sighted people decide to play it, making it smaller would be nice.

I love how simplistic it is and the sound effects for winning or losing. Also the color contrast is good. I know you said it's still in progress, so I would make getting the cards recognized by VO your main priority and then if you want to follow my other suggestions, you could.

natasha-v commented 4 years ago

Thank you so much for your feedback! Would you mind elaborating on what you mean by the cards not being recognized by VO? For me, I can "see" all the cards by either navigating down the page (VO-Arrow keys) or by invoking the rotor (VO-U).

bethany39 commented 4 years ago

I am testing on my iPhone and when I use the swipe gestures or just move my finger around, it never recognizes the cards. If I swipe, it goes directly from dealer to the player name.

Okay, now I can get to them sometimes by swiping up when rotor is on Lines, but it's not very consistent.

natasha-v commented 4 years ago

I see what you are talking about on an iPhone, and we will look into it further. Thanks!

natasha-v commented 4 years ago

Thanks again for your feedback! We've hopefully addressed this issue and are now looking into styling and an additional TTS option.

bethany39 commented 4 years ago

Yep, the cards show up now on iPhone!