tlchicken / ped Goto Github PK

According to the app info for the command set, the parameter for mc only should not be less than 0 which means 0 should be a valid input. However an error is shown.

There appears to be too wide of a gap between the year and the semester, possibly due to an extra space.

As a user of this application, it feels very unnatural to specify the module I am referring to by it's INDEX in the shown list, as I would have to search for it in the list first to see what the index is.

A more intuitive way may be to refer to a module by it's CODE instead since all modules have unique CODEs
i.e. take CODE y/YEAR s/SEMESTER

I'm not sure why but list has a noticeable lag when I use it with this data file, hope this helps in optimisation.

Append to the end of the data file:

{
"code" : "PHS1101",
"title" : "BILLION DOLLAR PILL",
"description" : "'''"""''""''"""""""""'",
"mc" : 99,
"academicYear" : 0,
"semester" : 0,
"tagged" : [ ]
}, {
"code" : "DMY1401TSG",
"title" : "Abc mc mod",
"description" : "smol",
"mc" : 1,
"academicYear" : 0,
"semester" : 0,
"tagged" : [ ]
}, {
"code" : "ACC1701X",
"title" : "Accounting",
"description" : "acc",
"mc" : 1,
"academicYear" : 1,
"semester" : 1,
"tagged" : [ ]
}, {
"code" : "CS3203",
"title" : "Abcde",
"description" : "abcde",
"mc" : 4,
"academicYear" : 0,
"semester" : 0,
"tagged" : [ "aaa", "ccc", "bbb", "eee", "ddd", "nnnnnnnn", "ggg", "fff", "iii", "hhh", "jjj", "mmmmmmmmmmm", "ertffgh", "wow", "age" ]
}, {
"code" : "1a2b3c4d5e6f7g8h9i110jb",
"title" : "Abcde",
"description" : "abcde",
"mc" : 4,
"academicYear" : 0,
"semester" : 0,
"tagged" : [ ]
} ]
}

Even though 1 MC is taken, it appears like no MC had been completed yet based on the bar.

A suggestion would be to make the bar minimally show a tiny red line if 1 or more MCs had been completed.

When all 3 parameters are specified for the set command, the error message a missing space between the word YEAR and the s/ prefix. (As shown by the red arrow in the screenshot)

When opening the app for the first time, a few modules have text that are too long to fit on the screen as shown in the screenshot. The module codes are then cut off which causes a lot of inconvenience as the module code uniquely identifies a module.

As a user, the module code and it's MC count is quite important (Especially the module code as it is the module's identifier) but I am unable to see their full text.

A suggestion would be to always show the module code and only allow the module name to be truncated.

The only requirement for MC is that it is an integer and must not be less than 0, so a large integer 99999999999 should be valid. However it does not work as seen in the screenshot.

When 1 quarter of the MCs are taken, the bar is not filled up proportionally.

The circled screenshot is on the same line as the text which makes the formatting look weird as the different pictures are not aligned with each other.

... the different commands only use the m prefix for MCs which only takes in integers

According to the User Guide as shown above, the edit command is supposed to work only if it is replacing a value with something new but in actual fact there is no error even when editing a module without new values.

According to user guide, the command add c/CS3203 t/Abcde d/abcde m/4 t/a should be a valid input as there is no restriction on tags but it does not work.

If a user wants to tag modules with his grade, tagging modules with single letters would be very useful and should work, however this input is rejected.

This is very inconvenient as the user would have to scroll back down.

This happens at least for take, untake, add.

There is a grammar issue in the command result for the clear command, where untaked is grammatically wrong.

Also there is an extra space in between the year and semester description.

As a user of this application, it feels very unnatural to specify the module I am referring to by it's INDEX in the shown list, as I would have to search for it in the list first to see what the index is.

A more intuitive way may be to refer to a module by it's CODE instead since all modules have unique CODEs
i.e. untake CODE