adityab4 / ped Goto Github PK

Running the clear command leaves the previous information in the information prompt.

It could be useful to clear out the information panel, even while the clear command is being run/incomplete.

Information panel states "Person type should only be 'stu' or 'ta'

The current user flow for delete is that the command shows a confirmation popup, before input validation. It might be beneficial to just do input validation before the popup, and simplify user flow by not having the user make a decision when the decision will be void.

On MacOS, when the Help section is opened using the TrAcker menu bar, the button next to the UG link has it's text truncated to Copy U....

Note that this only happens when the Help floating menu is in focus. When focus is on another window or application, the button text is no longer truncated.

Truncated, in focus

Not truncated, not in focus (clicked on this browser window before taking screenshot)

Inside the callout, under the Editing the data file sub-heading, the text reads "Caution: If your changes to the data file makes its format invalid, AddressBook will discard all data...".

This is the base products name, and could be changed to reflect TrAcker instead.

The user experience of the exit command could be removed by adding a short delay before closing the application so that the information panel text can be read.

Upon deleting a previously valid tag, all contacts with those tags added to them are updated, and all occurrences of the tag are removed. This cascading behaviour is not documented in the user guide and could potentially confuse users.

UG section for reference

On MacOS, when running the clear command, the two buttons are truncated, making it impossible to see what will be done.

The user guide title states "AB-3" instead of TrAcker.

Screenshot for quick reference

To reproduce

1. Add John Doe:

add stu /n John Doe /i A0123456Y /p 91234567 /e [email protected]

2. Add "JohnJohn DoeJohn DoeJohn DoeJohn DoeJohn..." with all the other details being the same

add stu /n JohnJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn DoeJohn Doe Doe /i A0123456Y /p 91234567 /e [email protected]

Output

Using an arbitrarily long tag like aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa causes the UI to break by pushing the label outside of view.

The add command allows arbitrary length inputs:

add stu /n JohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohnJohn  /i A0122256Y /p 91234567 /e [email protected]

This can pose a security risk, or cause unexpected memory usage