qwoprocks / pe Goto Github PK

As per title, refer to ss below. The example is CS2030 but the ss shows IS1103

As per title, refer to ss below

As per title. The result display shoud be "module" instead of "modules"

As per image below, although the module name is clearly alphanumeric, it is still not accepted by the system. Perhaps change it to "Only valid module codes will be accepted"

1. I understand that the bottom part of the module display panel is for Quiz display, but when there are no quizzes, it leaves a very jarring blank space, as per the below screenshot:

2. The zoom link cannot be fully seen when the display is too small/zoom link is too long, and also cannot be highlighted. The only place where it is stated that the zoom link can be copied by clicking on it is in the UG, and it seems that I have to double click to actually copy it (I tried by first clicking one zoom link then clicking another, only works for the second one when i double click). There is a lack of visual feedback in the application to indicate that the zoom link has been copied, and the result display remains as the previous command's result. I suggest changing the zoom link to a highlightable text field instead, and having a button or some other visual feedback when copying over the zoom link.

As per title. This might confuse the user, since there appears to be no trigger for the color change. Not sure if there's a reason for the color change. But the animation is pretty cool

Before:

After:

Even when a command has no actual effect, it still changes the color

Before running completetask 1 on the completed task 1:

After runnning:

Some of the commands (especially the ones that begin with new) are encapsulated with quote marks ' " ' while some aren't.

As per title. Refer to images below

Before running the addzoom command:

After running addzoom 2 n/lecture-weds z/https://nus-sg.zoom.us/j/auya7164hg:

Then immediately running viewmodule 2

As you can see, the module details display only updates after you call viewmodule, and not on any change to the module itself, which makes it very confusing for the user.

I have tested it out with other commands, and since it doesn't update with even the 'undo' and 'redo' commands, I'll be upgrading this severity to high. It is super confusing when you keep typing undo and nothing on the screen changes, not even the result display for the command.

As per title. The team should consider setting a minimum size for the window resize. Refer to screenshot below.

Making it slightly bigger also suddenly changes the size of the pie chart, which might be jarring to some users, as per the screenshot below.

Event should begin with a capital 'E', similar to previous issue.

Should the boolean be a box by its own? "boolean" is a primitive and thus I think it would be better to put it in the "Contact" box as a field. Also, the package names should have their first letters capitalized, e.g. "Contact" and "Tag", as per other UML diagrams.

As per title. There is a broken image link inside the UG pdf, maybe the team forgot to remove it? Refer to ss below

UG says 4 levels, but the app says 3 , refer to image below. Also, some grammar mistakes, e.g. it should be "levels" instead of "level" (in app), "will work fine" instead of "work fine" (in UG)

As per title. As seen in the ss below, the Grade part is obscured partially by the tags.

As per title. If I'm not wrong, it should be MODULE_NAME instead, since NAME is only for contacts. Refer to ss.

As per title. In the UG, the jar file name is given as CAP5Buddy.jar

But in the github release page, the jar file name is CS2103T-F12-3.CAP5BUDDY.jar

As per title. This is at page 25/114 in the DG pdf.

As per title. Normally, a user would expect the clearmodule command to clear everything on the screen, including the left display panel, but it still remains in this case.

As per title. The name of the ArchiveModuleCommand is "a" but "command" is returned instead. The "CommandResult" isn't given a name but "result" is returned from the ArchiveModuleCommand after the "execute" call. This is inconsistent with all DG diagrams that came before it.

Refer to image below. The team should consider wrapping the text, since truncating it would mean users have no way of viewing the full tag.

This also happens in the module view:

As per title.

As per title, refer to screenshot below

As per title. The one in the application is missing a space

As per title. Refer to the 2 screenshots below:

Short scrollbar:

Long scrollbar after scrolling up:

I think the dashed arrow is supposed to come from outside the Model box, as per the other diagrams in the DG.
Also, if the ReadOnlyUserPrefs is an interface, it should have the <> tag, and if it is not, the arrow pointing at it should be solid, not dashed.
Lastly, aggregation is not encouraged as per textbook, it would be good if it's changed to composition.

Refer to ss below. The module name only contains alphanumeric characters, yet it is rejected because of the small letter 's'. Changing it to uppercase causes the command to pass. The error message should be changed to reflect this constraint.

The image is updated but the description doesn't seem to be. Refer to two ss below where 1st image shows several components under MainWindow, but the 2nd (description) doesn't seem to reflect this fact, for example statusBar instead of StatusBarFooter, and the lack of the HelpWindow component. Also, under the 2nd image, the "API:" part is empty.

First image:

Second image:

As per title. The app cannot be seen as well.

Preference.json

As per title. See image below. Command desc is given as delete 1 but diagram shows deletemodule 1

As per title. See image below. There should be a space between # and the word after it. e.g. "...# Edit" and "...# Delete"

As per title. It can be seen that the scrollbar cuts off halfway, and the calendar is located at a weird position, with the right border cut off as well.

I realise that the calendar is located there because the top panel is reserved for module displays and stuff, but it makes it very confusing to the user.

First Aspect is blank? I'm not too sure if this is intentional.

The word "Cons" isn't separated from the previous line. Makes it look like everything is under Pros if you don't read properly.