wearable-learning-cloud-platform / wlcp-issues

0 stars 0 forks source link

Color Sequence Left/Right Buttons Clickability Indicator #239

Closed apoh3 closed 1 year ago

apoh3 commented 2 years ago

The left and right buttons for scrolling through color sequence input should be disabled if scrolling is not allowed in one direction. For example, if the user has scrolled all the way to the right and there are no more colors to be seen then the right button should be visibly disabled.

Related to issue #225

mmicciolo commented 2 years ago

@franciscastro @apoh3 @geseiche @IvonArroyo

Ready for testing at http://dev.wearablelearning.org/

I have updated the logic so that the left and right buttons are only enabled if you can scroll in that direction.

Testing Scenarios

The following scenarios should be tested and successfully pass for this issue to be marked complete and closed.

Scenario 1

The following scenario tests that the left and right indicators enable/disable correctly.

Steps for testing

  1. Create or load a game with a color sequence in a transition
  2. Debug the game
  3. At first left/right will be disabled since there are no sequences entered yet
  4. Adding 5 sequences the left button should enable
  5. You should then be able to scroll back 1 until the left button disables and the right button enables
  6. Scroll right again until right is disabled and left is enabled
  7. Add more sequences
  8. Scroll to the left and right to verify the buttons shut off when they should
  9. Scroll so that you are in the middle of the sequence you are currently entering.
  10. Add another color and it should jump to the end and disable the right button while leaving the left enabled
  11. Verify the above with more sequences
apoh3 commented 2 years ago

Test passed

Also checked if arrows disabled when sequence is cleared (worked).

Empty sequence: image

5 blocks in sequence: image image

In the middle of a long sequence: image

sai-educ commented 1 year ago

Test passed. Buttons behaving as expected. Please close this ticket.

Screen Shot 2022-09-14 at 9 04 57 PM Screen Shot 2022-09-14 at 9 04 51 PM Screen Shot 2022-09-14 at 9 04 39 PM

mmicciolo commented 1 year ago

Thank you @apoh3 and @sai-educ for testing.

Testing has passed.

Closing issue.