Skip to content
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

Open
wants to merge 546 commits into
base: master
Choose a base branch
from

Conversation

zhijietan94
Copy link

No description provided.

Copy link

@kychua kychua left a 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>
Copy link

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...`
Copy link

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...]`
Copy link

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.
Copy link

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`
Copy link

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.
Copy link

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>
Copy link

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?

@kychua kychua added the Reviewed label Oct 7, 2016
snowfeline and others added 27 commits October 28, 2016 09:21
…dBox.fxml, PersonListCard.fxml and PersonCard.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants