New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
OrderList and ScheduleList fxml #73
OrderList and ScheduleList fxml #73
Conversation
… into tutorial-adding-command
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
overall good job
<Label fx:id="orderId" styleClass="cell_small_label" text="\$orderId" /> | ||
<Label fx:id="customerName" styleClass="cell_small_label" text="\$customerName" /> | ||
<Label fx:id="phoneName" styleClass="cell_small_label" text="\$phoneName" /> | ||
<Label fx:id="phoneColour" styleClass="cell_small_label" text="\$phoneColour" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@uberSaiyan @qiujiaaa @zhixianggg Order for Model does not have Colour and Capacity but should we add it in? or we just use the phone in that order to access instead ?
I think we should stick with the latter as no phone means no order
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can just use the phone to access i think!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think accessing through phone is fine
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay stick with phone. Issue close
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
overall good job!
<Label fx:id="orderId" styleClass="cell_small_label" text="\$orderId" /> | ||
<Label fx:id="customerName" styleClass="cell_small_label" text="\$customerName" /> | ||
<Label fx:id="phoneName" styleClass="cell_small_label" text="\$phoneName" /> | ||
<Label fx:id="phoneColour" styleClass="cell_small_label" text="\$phoneColour" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think accessing through phone is fine
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's okay!
Added fxml for OrderListCard, OrderListPanel, ScheduleListCard and ScheduleListPanel.