search

zhing22 / pe

0 stars 0 forks source link

UI component image #8

Open zhing22 opened 2 years ago

zhing22 commented 2 years ago

should GuestListPanel and VendorListPanel be 0..1? (depends in your implementations)

Screenshot 2021-11-12 at 5.16.33 PM.png

nus-se-script commented 2 years ago

Team's Response

Thank you for your time and effort in writing this bug report.

This was intended and is actually not a bug. Implementation-wise, there is both one GuestListPanel and VendorListPanel, however, within the UI, there is a listPanelPlaceholder, that gets replaced by either panel, depending on the which one the application would like to render. However, instances of both still exist, just one gets hidden and the other gets shown.

You may refer to the implementation here: https://github.com/AY2122S1-CS2103T-W12-3/tp/blob/master/src/main/java/seedu/address/ui/MainWindow.java

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]