JunCheng98 / pe

0 stars 0 forks source link

Add option to close User Guide window after it is activated #2

Open JunCheng98 opened 3 years ago

JunCheng98 commented 3 years ago

After typing 'help', the User Guide window pops out for user convenience. Could be better if there was a button added to close that window after viewing, since some users may be confused as to how to close it and return to the main application.

nus-se-bot commented 3 years ago

Team's Response

After typing ‘help’, the User Guide window pops out for user convenience.

On the contrary, the user guide window does not pop out; it is embedded within the application itself.

Could be better if there was a button added to close that window after viewing, since some users may be confused as to how to close it and return to the main application.

It is impossible to add such a button as the window is embedded in the application; to close the window would be to close the entire application.

Immediately after the help command is introduced in the user guide, the tab command is introduced which the user can use to switch to any of the four tabs in the application. The user can use the tab command to switch to any of the tabs and exit the user guide. Furthermore, the user can click on any of the four tabs in the header of the application in order to switch to the respective tab.

There are also many other commands which will automatically switch the UI tab upon execution (e.g. add-expense, list-expense). The user can find out about the usage of these commands in the user guide, and then immediately execute the command while staying on the user guide tab page; the UI will automatically switch tab afterwards.

Since there are many ways for the user to exit the user guide, we have updated the severity from medium to low, as this would only pose a minor inconvenience the first time the user uses the application. In addition, the tester's suggestion of adding a button to close the window does not work due to the reason highlighted above.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]


:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: [replace this with your reason]