search

nus-cs2103-AY1920S2 / pe-dev-response

0 stars 0 forks source link

Missing CalendarPane method in DG #2575

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.

image.png

In Figure 19 of the DG, the method of CalendarPane which is called is absent.

[original: nus-cs2103-AY1920S2/pe-interim#2570]

alcen commented 4 years ago

Team's Response

The highlighted arrow does not have a method because there is no method called in the CalendarPane class. The CalendarPane contains an ObservableList, which has a listener listening to the changes made to the FilteredList in Model. Hence, to not confuse the reader, no method is included for that particular arrow.

However, we admit that it may lead to some loss of clarity and a description for the arrow could be included to make it more understandable.

Thank you for pointing this out!

Since this is just a cosmetic issue with the labels on the diagram arrows, we have decreased the severity accordingly

Duplicate status (if any):

--