SauveJeanLuc / coffee-tasks

Next gen AI Powered Todo-List

https://coffeetasks.netlify.app

MIT License

38 stars 73 forks source link

fix the confirmation dialog #148

Closed uKiJo closed 1 year ago

uKiJo commented 1 year ago

Description

Fixing the confirmation deletion dialog to make it conform to the design system of the application.

Related Issue

Motivation and Context

How Has This Been Tested?

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works

Screenshots (if appropriate):

netlify[bot] commented 1 year ago

Deploy Preview for coffeetasks ready!

Name	Link
Latest commit	e63c39cc97fb00b1a01f37cecbafb722cea15a92
Latest deploy log	https://app.netlify.com/sites/coffeetasks/deploys/635116819181f20007e19dda
Deploy Preview	https://deploy-preview-148--coffeetasks.netlify.app/
Preview on mobile	Toggle 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

This is a little bit lengthy

SauveJeanLuc commented 1 year ago

You can consider improving the positioning of the buttons with different screen sizes

SauveJeanLuc commented 1 year ago

It's not fixed yet @uKiJo

SauveJeanLuc commented 1 year ago

Replace "Yes I am sure " with Delete

SauveJeanLuc commented 1 year ago

Fix linting and we are good to go

SauveJeanLuc commented 1 year ago

Also check #62

uKiJo commented 1 year ago

Fix linting and we are good to go

i don't have those error when running the linter script...