nus-se-bot
commented
4 years ago Team's Response
This is actually the intended behaviour, we believe that if you add this restriction to the user, it will bring unnecessary inconvenience to the user as the user needs to use list before opening another flashcard.
It increases the user's productivity as he has to type less to achieve the same outcome (making the app more optimized for fast typists) and I don't really see how it's a source of confusion? It's like you are in a default list and just "scrolling" through the flashcards with the open command, it's pretty intuitive I believe.
Furthermore this is more of a suggestion and under PE guidelines testers should not be posting suggestions. You can refer to the image below.
Items for the Tester to Verify
:question: Issue response
Team chose [response.Rejected]
- [ ] I disagree
Reason for disagreement: [replace this with your explanation]
:question: Issue severity
Team chose [severity.VeryLow]
Originally [severity.Low]
- [ ] I disagree
Reason for disagreement: [replace this with your explanation]
I feel that an exception should be thrown telling the user that they have to use list to get back to all flash card before opening another one.
Example: open 2 while flashcard 1 is currently opened will directly open flashcard 2
Reasoning: Confusion