search

SauveJeanLuc / coffee-tasks

Next gen AI Powered Todo-List
https://coffeetasks.netlify.app
MIT License
38 stars 73 forks source link

feat: add a description field to the todo #139

Closed gaber85 closed 1 year ago

gaber85 commented 1 year ago

Description

This PR adds a description field, an optional field where the user can type important information about the task.

This field will follow the same pattern and behavior as the other existing inputs.

Related Issue

The related issue is https://github.com/SauveJeanLuc/coffee-tasks/issues/138.

Motivation and Context

Currently there is only a title and status, but no field where the user can write a description or notes about the task.

Checklist:

netlify[bot] commented 1 year ago

Deploy Preview for coffeetasks ready!

Name Link
Latest commit 14ad620201c66d904ec9f22542f175eae49a7203
Latest deploy log https://app.netlify.com/sites/coffeetasks/deploys/634faed24be4c60008ded96b
Deploy Preview https://deploy-preview-139--coffeetasks.netlify.app/
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

SauveJeanLuc commented 1 year ago
  1. Don't you think this option will ruin the app's simplicity?
  2. I can see the description is not optional at all, I can still see the input text wether I want to put it or not.
gaber85 commented 1 year ago

@SauveJeanLuc Not at all, in fact it adds value to the users that want to add more info about the task. It still keeps the simplicity. No worries if you don't feel comfortable adding this feature. After all you are directing how this app should be! I can make tweaks if you like the idea.

The optional aspect of the field is that you don't need to add text to it if you don't want to, unlike the title, where you cannot leave it blank and is required.

SauveJeanLuc commented 1 year ago

Great, go on with the tweaks in a way that maintains the app's simplicity. Like I don't have to see the description textbox, if I'm not really willing to add the description

gaber85 commented 1 year ago

@SauveJeanLuc that sounds good, I'll make the changes and push it back up!

gaber85 commented 1 year ago

Hey @SauveJeanLuc what do you think of this? I've attached a demo! I've pulled master and it's ready to merge if you like it!

https://user-images.githubusercontent.com/20451159/196632591-9c3d9660-4dd7-41f6-9e6e-a77c1f805879.mov