Skip to content

Make setting due date today an option in context menu - #686

Merged
diegogangl merged 1 commit into
getting-things-gnome:masterfrom
syvb:start-due-today
Jul 21, 2021
Merged

diegogangl merged 1 commit into
getting-things-gnome:masterfrom
syvb:start-due-today

Conversation

@syvb

@syvb syvb commented Jun 30, 2021

Copy link
Copy Markdown
Contributor

This is useful for when you want to reschedule future tasks to today, or bump forward overdue tasks to today.

@nekohayo

Copy link
Copy Markdown
Member

Hi, had you seen issue #550? There's some nuances in there, where I thought, regarding the start dates:

"Today" [as a start date] makes little sense when you have "Clear Date" and would suggest removing it (and the separator that follows it) [from the popover menu from the headerbar] for consistency [with the right-click menus].

For the due dates however, I agree that "Today" is missing from the right-click menu. I didn't think it would be a huge deal when I saw it being removed last year, but I can definitely say that in practice it happens more often than not that I want to quickly retarget the due date of a task to today...

This is useful for when you want to reschedule future tasks to today, or
bump forward overdue tasks to today.
@syvb
syvb force-pushed the start-due-today branch from e6a7db4 to 278cc04 Compare July 12, 2021 20:15
@syvb syvb changed the title Make setting start/due date today an option in context menu Make setting due date today an option in context menu Jul 12, 2021
@syvb

syvb commented Jul 12, 2021

Copy link
Copy Markdown
Contributor Author

@nekohayo I've updated the PR to just change the due date menu

@Neui Neui left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks ok code-wise and works.

@diegogangl
diegogangl merged commit ed099fe into getting-things-gnome:master Jul 21, 2021
@diegogangl

Copy link
Copy Markdown
Contributor

Merged, thanks!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants