yanbingtao / ped Goto Github PK

You may want to resize the message feedback pane, as current version couldn't display the full feedback message when the message is really long.

For example, in the pic below, the feedback message should also be able to display the missing part, which is "311, Clementi Ave 2, #2-25 Tags: [owesMoney][friends]" You may refer to the pic below. Thanks.

The GUI has been improved, but seems like sime tabs coundn't display the full name (may refer to the pic below). You may rearrange the tab size to display the full name so that users could know what the tab is for? Thanks

In the patient list, it didn't show what does each information is able. For example, the first patient - Alex, the date "2019-02-28" should be the birth date. But there's no subtitle to tell the user that this is for birth date. Users also could interpret as "the appointment date" or other kinds of date? same for blood type, it shows "A+" for alex, but didn't really tell the user that this is to record the blood type.

Maybe you can consider to add a lebel in front of each data. For example, instead of displaying "MALE", can show "Gender: MALE". It will be clearer.

As shown in the pic below, in the schedule panel, the text color for weekday's date is very close to the background color, cannot really see the texts. You may consider to change the color for this part. Thanks.

As shown in the 1st pic below, the appointment is on 25 Jan 2020, but when i command c-list (supposed to show only upcoming appointments, refer to 2nd screenshot from UG) on 30 Oct 2020, the application will still show the appointments in Jan 2020, which is already in the past (not upcoming). So maybe can check the current date before displaying out? Thanks

In the UG, it shows that to schedule an appointment, users should enter command like this a-schedule i/2 start/2020-09-14 08:00 end/2020-09-14 10:00 d/Review Appointment(shown in 1st pic), but when I copied and pasted this command over to the application, it shows invalid command error (shown in 2nd pic).

The command box is not in the correct size. some part is covered by the Patient list. So when the command is very long, some texts will be covered by the patient list, cannot be shown. You may want to change this part.

In the UG, it states that the format for c-edit command should be p-edit INDEX [n/NAME] [p/PHONE_NUMBER] [e/EMAIL] [a/ADDRESS] [t/TAG]…​

But in the GUI, it shows p-edit INDEX n/NAME p/PHONE_NUMBER e/EMAIL a/ADDRESS t/TAG, which contradicts with the version in UG. In the GUI version seems like users must provide all these info (NAME, PHONE_NUMBER, EMAIL, ADDRESS, TAG). May be you want to add [ xxx ] to show that all these info are optional. Thanks