dhruvasagar / vim-dotoo

Org-mode like task logging & time tracking in Vim
558 stars 27 forks source link

@dhruvasagar Hey there, hope you don't mind if I chime in. I'm also another user who was looking forward to this change, and am very excited to see this implemented. The few suggestions I have are: #99

Closed dhruvasagar closed 3 years ago

dhruvasagar commented 3 years ago

@dhruvasagar Hey there, hope you don't mind if I chime in. I'm also another user who was looking forward to this change, and am very excited to see this implemented. The few suggestions I have are:

Originally posted by @ourigen in https://github.com/dhruvasagar/vim-dotoo/issues/92#issuecomment-798925370

dhruvasagar commented 3 years ago

@ourigen I have address the first part of your request, kindly refer to the documentation / commit (c6dcbcd) description for details.

Addressing the second part is also easy, I am thinking about any side effects. Will share here once I push something.

dhruvasagar commented 3 years ago

@ourigen I have addressed both these requests. Kindly take a look and let me know.

ourigen commented 3 years ago

@dhruvasagar Thanks for the quick work on this! From my brief test, both features are taking effect and working well. However, tasks dated for the future are repeated under the current day in addition to on the deadline/scheduled day. Is it possible to show the tasks only once on its associated date?

dhruvasagar commented 3 years ago

@ourigen That is by design as I understand, for today we try to show all upcoming tasks, which includes tasks in the future. There's a separate configuration that controls how far in future we should look. I will still refer to the org mode documentation in this and see if I find any discrepency.

dhruvasagar commented 3 years ago

@ourigen perhaps in week / month span modes, this rule shouldn't apply, I will cross check with Org Mode on this and revert.

ourigen commented 3 years ago

@dhruvasagar Now that you mention it, I do see the purpose of having tasks also listed under the current date. Honestly if it is easier to leave it as is, I wouldn't mind. The main features I was looking for were already quickly implemented, so thank you for your hard work on this