-
Notifications
You must be signed in to change notification settings - Fork 6
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
Convert mock ui into code #100
Conversation
Only added one screen so far and it works only for person class currently.
switching tabs currently is not yet changing the cards display.
…into ui � Conflicts: � src/main/java/seedu/address/ui/MainWindow.java
…into ui � Conflicts: � src/main/java/seedu/address/ui/MainWindow.java
import seedu.address.ui.textstyle.TitleDescription; | ||
|
||
/** | ||
* An UI component that displays information of a {@code Person}. |
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.
Really minor a UI not an UI
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.
The other JD comments are in the other PR, you can ignore them for now if you want unless you it is finalized
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.
Have fixed the java docs issues, thanks.
No description provided.