fix(TaskBody): show due date instead of start date in task list#3190
Merged
raimund-schluessler merged 1 commit intoJun 29, 2026
Merged
Conversation
The recurring tasks change (nextcloud#3021) made the task list prefer the start date over the due date whenever a start date was set, so a task with both a start and a due date wrongly showed its start date and hid the due date (nextcloud#3187). Revert the date column to its pre-0.18.0 behavior: always show the due date. The start date is not the actionable date in the list, and showing it in the same column as the due date - same style, no label - made it impossible to tell which date was shown. Recurrence does not change this: the completion handler advances the due date (anchored on `task.due || task.start`), so the due date is the relevant value for recurring tasks too. The recurring indicator icon is unaffected. This also restores correct overdue semantics: a task is overdue based on its due date only, not its start date. Fixes nextcloud#3187
Member
|
@schulm Thanks a lot for the comprehensive explanation and for providing the fix! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fix: task list shows due date instead of start date (#3187)
Problem
Since the recurring-tasks change (#3021), the task list started showing the start date instead of the due date for tasks that have both set — a regression from 0.17.1 (#3187).
Background
I tried to remember the reasoning behind the original change. As best I can reconstruct it, the idea was to surface some date for tasks that have only a start date (which previously showed nothing) — but the precedence ended up backwards, so the start date was preferred in all cases, hiding the due date whenever both were set.
Thinking it through again: the start date isn't the actionable date in the list, and recurrence doesn't change that — the due date is what matters, and it's also the value the recurrence completion advances (
task.due || task.start). Putting a start date in the same unlabeled column as due dates was also the source of the "which date is this?" confusion.Fix
Revert the date column to its pre-0.18.0 behavior: always show the due date.
v-if="task.due || task.completed"dueDateShort/dueDateLonguse the due date directlyThe recurring indicator icon is unaffected. Start-only tasks show no date text (the long-standing behavior) and still get the existing
CalendarClockicon hint.Testing
npm run lint,npm run dev, and the unit test suite (vitest) all pass.Fixes #3187