Skip to content

fix: restore subgroup indentation in TaskList views#1788

Merged
callumalpass merged 39 commits into
callumalpass:mainfrom
renatomen:fix/tasklist-subgroup-indentation
Apr 26, 2026
Merged

fix: restore subgroup indentation in TaskList views#1788
callumalpass merged 39 commits into
callumalpass:mainfrom
renatomen:fix/tasklist-subgroup-indentation

Conversation

@renatomen
Copy link
Copy Markdown
Contributor

Description

Fixes subgroup headers not displaying with proper indentation in TaskList views when sub-grouping is configured.

Problem

When viewing tasks with both primary grouping and sub-grouping enabled, sub-headers were rendering at the same level as primary headers without visual indentation, making the hierarchy unclear.

Before: Sub-headers appeared flush left with primary headers
After: Sub-headers display with 16px left padding showing clear hierarchy

Solution

Added CSS rules for [data-level="sub"] attribute selector in task-list-view.css:

  • 16px left padding for indentation
  • Smaller font size (0.95em) and muted color for visual distinction
  • Reduced opacity and size for chevron icons on sub-headers

The TypeScript code was already setting data-level="sub" on sub-header elements, but the corresponding CSS rules only existed in the Bases-specific stylesheet. This PR adds the missing rules to the general TaskList view styles.

martin-forge and others added 30 commits April 5, 2026 01:01
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…d date (callumalpass#396)

toggleRecurringTaskComplete and toggleRecurringTaskSkipped now default to
the task's scheduled occurrence date for scheduled-anchor recurring tasks
when no explicit date is passed, instead of unconditionally using today.
…ass#396)

The plugin-level toggleRecurringTaskComplete wrapper independently
derived the completion date from getTodayLocal(), diverging from the
TaskService fix. Now uses the same scheduled-anchor resolution.

Also replaced pseudo-test with a behavioural test that exercises the
real TaskService.toggleRecurringTaskComplete method with mocked
dependencies.
…sts (callumalpass#396)

The main.ts wrapper was resolving the notice date from the caller's
potentially stale task object. Now reads from cacheManager before the
service call, matching the authoritative source the service uses.

Added behavioural regression tests for toggleRecurringTaskSkipped to
match the existing completion path coverage.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Make the current Tab key behavior of focusing the next input field in the details part of the task modal toggleable.
When this option is disabled, pressing Tab triggers the default behavior (e.g. indenting checkbox items)
Subgroup header indentation CSS selectors had a typo
(tn-tasknoteTaskList instead of tn-tasknotesTaskList) causing the
rules to never match.

Also added margin-left to task cards under subgroups so their
chevrons align directly below the subgroup header chevron.
…endar-sync-reliability

Fix Google Calendar sync race condition and recurring completion date bug
…65-auto-archive-calendar-cleanup

Fix auto-archive Google Calendar cleanup retries
@callumalpass
Copy link
Copy Markdown
Owner

Hey @renatomen ! Hope you've been keeping well. Thanks for this--merging for the next release :)

@callumalpass callumalpass merged commit 6b6e5dd into callumalpass:main Apr 26, 2026
4 checks passed
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.

5 participants