search

ghos7ie / pe

0 stars 0 forks source link

Violation of Constraint: Typing First #2

Open ghos7ie opened 1 week ago

ghos7ie commented 1 week ago

image.png

Continuing from issue 3.

There is no easy way to change the displayed list to show events, instead of persons. I have to press tab 4 times in order to reach the Event button from the command input. Even then it is not obvious enough that I am on the button.

nus-se-bot commented 1 week ago

Team's Response

Thanks for submitting this report.

Comments from dev team: Invalid

Bug Type that we chose: Feature flaw

Severity that we chose: Low

Decision that we made: This is under planned enhancement 8

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: I believe the response should be agreed.

This issue should have been fixed by v1.6 not a future iteration of their application since it violates the constraints laid out by the course, as mentioned here, in particular CLI-first and Typing Preferred.

## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** Since this constraint was clearly stated at the start of the tP, I believe it should be medium rather than low. It affects the normal operation of the app which should allow the user to do **everything** by typing, only offering clicking as a QOL feature. Having to click on the Person/Event tab ruins this flow and should have been a priority for the group to solve since it is part of their user profile. ![image.png](https://raw.githubusercontent.com/ghos7ie/pe/main/files/305c237f-ee22-4b14-a80a-b7a5ade2db86.png) They have actively ignored their user's profile by having users click on the tabs rather than implementing list commands. But since it does not break the application completely, this issue cannot be assigned a high. I originally reported this bug as an extension of [issue #3](https://github.com/ghos7ie/pe/issues/3) because the team's planned enhancement section contained too many enhancements. Keyboard shortcut/Command to swtich views was 8th on their list, but I believed should have been enhancements numbered 16 or 17 after counting their enhancements. So simply stating that they had this in their planned enhancement is not a convincing enough argument to me since this issue should have been their number 1 priority to solve.