-
Notifications
You must be signed in to change notification settings - Fork 109
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
[T7][T11-C3] #82
base: master
Are you sure you want to change the base?
[T7][T11-C3] #82
Conversation
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.
@zhijietan94 @GitMeGet @snowfeline @hongyuhy Some comments added. Please close the PR after reading comments.
- remember to acknowledge the original source of code and include a mockup of the UI in README.md
- pls tag V0.1
- make sure you've thought thought the UI - how do you plan to display deadlines and events? how are completed tasks displayed?
Format: `add TASKNAME d/TASK_DESCRIPTION @/DATE STARTTIME ENDTIME [t/TAG...]` <br> | ||
Format: `add TASKNAME d/TASK_DESCRIPTION @/STARTDATE ENDDATE TIME [t/TAG...]` <br> | ||
|
||
\*\* <i>Refer to appendix for Possible formats for DATE & TIME</i> |
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.
Where is the appendix?
Format: `add NAME p/PHONE_NUMBER e/EMAIL a/ADDRESS [t/TAG]...` | ||
#### Adding a task: `add` | ||
Adds a floating task to DoMePlease<br> | ||
Format: `add TASKNAME d/TASK_DESCRIPTION t/TAG...` |
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.
task description is compulsory? is that necessary?
Format: `add TASKNAME d/TASK_DESCRIPTION t/TAG...` | ||
|
||
Adds a deadline to DoMePlease<br> | ||
Format: `add TASKNAME d/TASK_DESCRIPTION @/DATE TIME [t/TAG...]` |
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 I have a deadline without a time?
* Edit the 2nd floating task in DoMePlease. Example Buy Pencil. | ||
* The format whereby the 2nd floating task was entered by user in the past will be populated on the command line. | ||
* `add Buy pencil d/Pencil to shade OAS sheet` <br> | ||
* User will edit the inputs accordingly and submit the edit field. |
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.
This means the user enters 'edit 2', enter, then makes the changes? If so, user takes multiple steps, and might take a lot of time to edit if say, the user wants to edit something near the end. Is it possible to make edit
easier to use? e.g. allowing user to edit a field directly
|
||
#### Navigating through Calendar: `select` | ||
Populate the list of deadlines and events on the input DATA of current month on the calendar panel <br> | ||
Format: `select DATE` |
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.
consider shorter commands e.g. view to reduce amount of typing by users
|
||
Examples: | ||
* `select 5` <br> | ||
* Assume that the calendar panel on the application is on the month of June. |
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.
What if I want to view another month?
* Output the full list of deadlines on the day and events that start, ends or is on-going on 5th June. | ||
|
||
#### Deleting a task : `delete` | ||
Deletes the specified task from DoMePlease. Irreversible <br> |
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.
irreversible meaning undo doesn't work on delete?
Merge show-save branch
Merge extend-undo-command branch
updated format of AboutUs.md
…dBox.fxml, PersonListCard.fxml and PersonCard.java
Merge sampledata branch
change online help to offline command summary
refactor code quality
No description provided.