show msgbox with confirmation del/clear

ajaxray / geek-life

The Todo List / Task Manager for Geeks in command line

MIT License

501 stars 26 forks source link

show msgbox with confirmation del/clear #35

Closed saaremaa closed 2 years ago

saaremaa commented 2 years ago

Added a confirmation window when deleting a project or clean the tasks.

fix: https://github.com/ajaxray/geek-life/issues/28 fix: https://github.com/ajaxray/geek-life/issues/21

2021-12-01_21-03-50 2021-12-01_21-04-18

ajaxray commented 2 years ago

Hi @saaremaa,

Thanks for the awesome feature! 🤩

Just a few points to discuss:

I've removed #8 from the fixing list because it's related to deleting an individual task.

saaremaa commented 2 years ago

My mistake - you are right. https://github.com/ajaxray/Geek-Life/issues/8 must be removed.

ajaxray commented 2 years ago

Hi @saaremaa ,

Thanks again for the awesome addition!

I have done a little refactoring in your implementation of showing modal confirmation. Can you please have a look and let me know if what I've done is Okay?