kalsyc / ped Goto Github PK

Before:

After adding the 15th exercise:

After adding the 15th exercise, the Pie Chart Legend disappears and can't be found. There are also no percentages shown for each exercise which makes me think that the feature is incomplete and hard to use.

Suggestion: Make the legend scrollable so that I can tell which part of the Pie Chart is referring to which exercise. It might also be better to have the pie chart bigger so that it's easier to see. Also, have percentages or some measure of some statistic so that the feature will be more complete.

The examples here in the UG do not work properly as Add command takes in mandatory parameters (/c, /q, /u).
Add command does not work when those parameters are missing.

Suggestion: Remove those negative examples and include more variations of proper examples.

2 regimes of the same name (1 scheduled, 1 in the regime list) cannot be resolved properly as the command Resolve works via indicating which regime is to be resolved.

Suggestion: Have indices or tags that mark out which regime and have the command resolve scheduling conflicts via those parameters. Also, it would be good to mark out which regime is the conflicting one and which regime is the currently scheduled one. Another way is to simply not allow 2 regimes of the same name to be created.

When an exercise of a long name (Let's say Inclined Bench Press with Smith Machine) is added into ExerHealth, the full name is truncated and the cut-off is shown as ellipses.

Suggestion: Either place a limit on the number of characters in the name of the exercise or extend the card that holds the name such that it wraps around a text box. We can also simply lengthen the card holding the text or decreasing the font size to ensure a long name fits.

User Guide and Feedback from system doesn't match. It is likely that the User Guide is wrong as the /t parameter is not required since edit only works for exercises, and the command works just fine without the /t parameter.

Suggestion: Decide on a consistent command format.

The error message shown for Add command is inaccurate as it tells me that there are no parameters for the command to add exercises into the exercise list. However, the error/help message does tell me the correct parameters for adding regimes into the regime list.

Suggestion: Give examples in the error message/fill in the required parameters for Add command for adding exercises.

Error message is wrong when attempting to create a regime that has a name that already exists.

Also, it is actually possible to create regimes of the same name if one has already been scheduled, which can lead to a potential functionality bug whereby 2 conflicitng schedules have the same name.

Suggestion: Should flag out that there already exists a regime of the same name.

The List Command format in the UG is incorrect as it does not have mandatory start and end date parameters.

Suggestion: Remove the start and end date parameters.

When viewing statistics for 31 days, the second day and the second last day is missing. This only happens when the range is 31 days.

Suggestion: Make those 2 dates appear.

Upon deleting of the first exercise and then undoing it subsequently, the exercise does not get added back to the original index but instead gets appended to the end of the exercise list. It does not break the app but it does not undo as intended.

Suggestion: Maintain a sorting order for the exercises or the item contains some index that allows it to be written back into the same location via undo.

Long names can cause the Bar Graph to disappear.

Suggestion: Either have a limit for the length of the names of exercises or make it truncate such that it only pops up when I hover over it.