Toggle issue fixed, Merge after Zoe pushes up her branch

CompassSoftware / AppTAG

AppState's TAG - The Accounting Game

MIT License

0 stars 0 forks source link

Toggle issue fixed, Merge after Zoe pushes up her branch #208

Open JerichoSM opened 4 years ago

JerichoSM commented 4 years ago

No images yet, but the underlying issue with the button not working is fixed.

JerichoSM commented 4 years ago

Working correctly, the "n" key should toggle the music on and off with a single click. musicToggle is now a global variable, in order to fix the issue of some rooms not being consistent, and still requiring and extra button press to turn off music.

Going forward, a music toggle icon should appear in the right side of the screen, small enough to not distract or clutter the view, but big enough to be noticed for utility.

fenwickjb commented 4 years ago

Looks great! Couple of hopefully quick fix suggestions:

Can the red lines be a bit "thicker"??
Can we move the image placement to the corner just above the coin count? Our focus is up there for "info" it seems.
When I started the game the music was initially playing but the icon was X'd out. After toggling with n it was "in sync"