lfos / calcurse

A text-based calendar and scheduling application
https://calcurse.org
BSD 2-Clause "Simplified" License
990 stars 94 forks source link

Delete a note for a TODO item #474

Open Shamaoke opened 1 year ago

Shamaoke commented 1 year ago

Add an ability to delete a note for a TODO list item. Can be implemented in the same way as deleting an appointment note, e. g.: select a list itempress da selection prompt appears asking you to either delete the (s)elected list item or delete a (n)ote for itpress nthe note will be deleted.

gaurav-cheema commented 1 year ago

Thats already a feature, no?

Shamaoke commented 1 year ago

No, it's not. When pressing d on a TODO list item, the prompt only asks you, if you want to delete the item. It doesn't ask you, if you want to delete a note for a given item.

gaurav-cheema commented 1 year ago

This is what I am getting when I try to delete a todo with a note. The option to delete the todo or only the note occurs for me after the y/n prompt to delete the task.

Unless I'm misunderstanding the TODO list youre mentioning, or have a different version. Screenshot_20230930_003216

Shamaoke commented 1 year ago

I have the 4.8.1 version of Calcurse. What version do you have?

gaurav-cheema commented 1 year ago

4.8.0 but yeah i do get the option

Shamaoke commented 1 year ago

The option to delete the todo or only the note occurs for me after the y/n prompt to delete the task.

This prompt misled me. Yes, I have that option too. But nevertheless, the process of deleting list items and notes should be consistent both for the list of appointments and the list of TODOs. This is not the case now.

gaurav-cheema commented 1 year ago

It's not too hard. I think Just flipping the prompts around would work.