soc-se-bot
commented
1 year ago Team's Response
This is a bonus feature purely for cosmetic purposes. The rest of the core features all use CLI.
Items for the Tester to Verify
:question: Issue response
Team chose [response.Rejected]
- [x] I disagree
Reason for disagreement: I disagree with this because although this feature is for cosmetic purposes, it is still a feature that is part of the product. This can be argued further that there are features in the app that is likely to be used less than changing themes but still has a CLI command.
For example, if a user prefers dark mode, every time the user enters the application, they would have to change it to dark mode (because it restarts to light mode by each launch), hence they would have to use the GUI elements to toggle this theme. On the other hand, every time a user wants to access the help page, they can type help, even if it is most likely that this feature is used less than toggling the themes. So depending on the preference of the user, I think that this feature can be considered important to a user.
To further reiterate, from the textbook, feature flaws are 'features that are not optimized for 'fast typists'.
I also would like to acknowledge that this bug is badly titled but I think it still brings up a relevant point.
## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** As because themes of the product aren't saved, it can be a hindrance to fast typists who prefer dark mode, to always use the GUI elements to toggle between themes which is why I think it should be severity.Medium.
The target audience for tp project is meant for people who are fast typers and prefer CLI. However, alternating to different view modes for dark and light mode has to have the user use the GUI elements.