se701g2 / Doto

Todo application for Group 2
https://doto.azurewebsites.net/
MIT License
12 stars 42 forks source link

issue/273 - Tasks do not show due dates #299

Closed Kalashnikkov closed 4 years ago

Kalashnikkov commented 4 years ago

Resolves issue #273 (Small typo on branch name).

Updates the CalenderComponent's modal to also display the Due date for a given task, as it's information is not used inside of Doto, an automatic scheduling app.

Current Behaviour: image

Proposed Behaviour: image

Extra:

There seems to be another bug where:

  1. Adding a fresh task will populate the dueDate field with a Date object.
  2. After refreshing the page, this dueDate is then updated to a String.

This however, does not affect this PR as functionality is not broken - but worth investigating.

preetpatel commented 4 years ago

Is there any way to left align that due date @Kalashnikkov ? Also I dunno how i feel about that red among all black. Makes it seem like an error

Kalashnikkov commented 4 years ago

@PreetPatel I couldn't find an elegant solution to left-aligning the text, as the styling is a bit different - so I just opted for centering the text instead.

jordansimsmith commented 4 years ago

Ideally, it would fit in with the 3:30 - 4:30 PM date in terms of style. I agree with preet about the red / centering.

preetpatel commented 4 years ago

merging as no action was taken on changing style..