Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add command: "Fold to tasks due next" #346

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

shalecraig
Copy link

@shalecraig shalecraig commented Jun 1, 2017

I usually try to work on the next task that's due.

This is easy to do when I have some overdue/upcoming tasks -- just use Tasks: Fold to due tasks, and select the most-overdue tasks first.

However, this isn't as easy when I'm ahead of myself (I try to stay ahead of my due dates). Because of this, I try to grab things that I don't need to do until then.

Implementation/Decisions.

  • Shamelessly, I copy-pasted most of the due_soon implementation, collecting "interesting" (aka parsed) (due timestamp, line) tuples, and sorted/took the first 10 at the end.
  • There's a TODO in there about setting up a sample/default value -- I'm not sure if there's a Default settings file I need to add the new settings to.
  • Most of the settings for the next_tasks fallback to the due_soon values instead of having their own defaults.

When added to the menu, this will fold tasks so the only ones visible are the 10 ones due next.
@vovkkk
Copy link
Collaborator

vovkkk commented Jun 1, 2017

Why not make it a default behaviour?
E.g. if there are no overdue and soon, but there are some next, we could ask user should it fold to next or not (with ok_cancel_dialog).
And there might be a setting for dialogue: ask, use next without asking, no asking and no folding.

@shalecraig
Copy link
Author

shalecraig commented Jun 1, 2017

Hmm, this is some good feedback.

Let me see what I can come up with -- it might not be immediate, but in a few days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants