ransome1 / sleek

todo.txt manager for Linux, Windows and MacOS, free and open-source (FOSS)
https://github.com/ransome1/sleek/wiki
MIT License
1.34k stars 104 forks source link

Clicking on a threshold date and choosing a date in the date picker results in adding a due date to the task #462

Closed loopernow closed 1 year ago

loopernow commented 1 year ago

Clicking on a threshold date and choosing a date in the date picker results in adding a due date to the task, rather than in updating the threshold date.

Steps to reproduce the behavior:

  1. Create a task with a threshold date, such as "t:2023-02-06".
  2. Click on the task to update it.
  3. The threshold date will be listed in the dialog box with a clock symbol. Click on the threshold date or on the symbol.
  4. Choose a date in the date picker.
  5. The task will now have a due date that you selected. The threshold date remains unchanged.

No errors seen in the Developer Tools console.

Expected behavior Clicking on the threshold date in the task dialog box and then choosing a date in the date picker should result in an updated threshold date.

Screenshots This is the relevant portion of the task dialog box: image

Desktop (please complete the following information):

ransome1 commented 1 year ago

Thanks for the bug report. I'm currently afk but will look into it as soon as possible.

ransome1 commented 1 year ago

@loopernow did you see there is are Due and Threshold tabs? You need to select which date type you want to change. The UX here is not ideal, I need to admit. There have been other complains too. However I don't think this is a bug.

grafik

loopernow commented 1 year ago

@ransome1 No, I didn't see that! Thank you.

Now that I know that, the part that still seems bug-like to me is that I'm clicking on the clock icon for a Threshold date, but am taken to a calendar for a Due date.

I can see how maybe it is not a bug, though. It is up to you!

aubreyz commented 1 year ago

I am not using Slick right now but likely will do when it is mature enough (as simpletask on android). However I do follow the github thread avidly for that reason. With that in mind, this does seem like a bug to me (or at least is an interface issue that is likely to lead to errors).

ransome1 commented 1 year ago

@ransome1 No, I didn't see that! Thank you.

Now that I know that, the part that still seems bug-like to me is that I'm clicking on the clock icon for a Threshold date, but am taken to a calendar for a Due date.

I can see how maybe it is not a bug, though. It is up to you!

I'm currently rewriting sleek. You can check out the latest developer preview. It's not meant for actual production use yet, but will give you an idea on how the the due and threshold dates will be separated in the future. It's basically about separate Date-pickers for both dates.

ransome1 commented 1 year ago

Will be dealt with in https://github.com/ransome1/sleek/issues/234

Let's close here.