Closed aidengeorge97 closed 5 years ago
Thanks for the feedback, I've made those tweaks!
Rebase this one (sorry!) and I'd be happy to approve.
This branch had too many conflicts to reasonably merge so I started from scratch on a separate branch. See #439 for the new PR.
Closes #368.
This PR serves to address interface issue users have identified in testing. The key issues were
To address the second point, I have made it so that the most recently added course now appears at the top of the list. This makes it clear that a course has been added, at the cost of not keeping the list in a sorted ordering. To mitigate this cost I have added an additional tool button which allows the users to sort the list by course code. These changes will allow users to immediately see when a course is added, while still being able to easily search through existing lists.
To address the first point I have changed the wording of the button and reformatted it to match the other added buttons. I believe the size/placement of the original button and its wording was overselling its importance to the addition of courses. This may still require further testing, but I feel user confusion comes more from unfamiliarity with the List functionality within the system as a whole.
Lastly I have added the ability to expand/collapse course lists to keep the whole user interface clean.