Personal Usability Test Results

[blarosen95]

Personal usability test notes:

• (I2-E1) Changing seasons will almost always shift the label and input for #changing season either left or right.
  • This is unsightly as the user's visual focus is immediately upon the pair when it shifts.
• (I2-E2) Changing seasons will also almost always shift the grid of episode checkboxes either left or right.
  • This is less unsightly, the user's visual focus is not immediately upon the component that shifts.
  • Would still like a fix but will watch other testers to determine value first.
• (I2-E3) Checkboxes selected do not remain selected (evident when changing back to their seasons)
  • Solving this could simplify input validation with the current concept planned for keeping an array in state with all selected episodes for a given show.
    • Fix this before beginning implementation of the array concept, then ensure that the array won't push a value that has already been checked when navigating among seasons.
      • (I2-E3.5) Do ensure, however, that the act of unchecking an episode removes it from the array.
• (I2-E4) I failed to fix the Lock Show checkbox with regards to handling uncheck events properly. Should be an easy fix.
• (I2-E1 but with emphasis) The issue with the inputs moving between season changes is really bad.
• (I2-E5 but better off pulled into new issue AFTER first round of the 3 usability tests are patched) There's still some console-logged pieces to handle (ranging from debug output to CSS and JS warnings/errors).

_Originally posted by @blarosen95 in https://github.com/blarosen95/Streaming_Playlists/issues/1#issuecomment-910681304_

[blarosen95]

I2-E4 resolved and passed a few quick, basic tests for functionality. Do note that we will need to pass a given show's selected episodes array back to the Show component as it creates a different Episode component instance when re-enabled (passing a function from Show to Episodes as a prop should work).

There's some counter logic involved in this, though: The record of the episodes back in the Shows (eventually to be called "Show" as I will have a new component rendering multiple Show components and that's best named "Shows" itself) component should not be submitted as part of the playlist if that showName value is not in the final form data upon submission. This presents some business logic challenges but I'll draw up two or more DMAIC flowcharts/diagrams documenting the process involved in both the user creating a playlist and the application saving a user's playlist, in order to establish a better concept here.

[blarosen95]

I2-E1 resolved and changes were made to account for #3 .

[blarosen95]

I2-E5 is mostly patched for now. Some CSS warnings remain but appear to be outside the scope of this feature itself, and not usability related at the moment.

[blarosen95]

I2-E2 is patched but the new design therein requires heavy UX feedback. The labels for episodes in the grid now wrap as needed. There is also now just enough margin involved to nearly perfectly align the central column at its checkbox items with the checkbox above for locking shows.

[blarosen95]

I2-E3 concerns logic more than is in-scope for approving the PR to implement the rough draft into master after I get at least one more usability test in from someone new. Closing this issue now and committing patches made between this issue and #3.