Skip to content

feat(widget): focus overflowing task lists - #42

Open
xz-dev wants to merge 3 commits into
tintinweb:masterfrom
xz-dev:feat/task-widget-focus-window
Open

feat(widget): focus overflowing task lists#42
xz-dev wants to merge 3 commits into
tintinweb:masterfrom
xz-dev:feat/task-widget-focus-window

Conversation

@xz-dev

@xz-dev xz-dev commented Jul 25, 2026

Copy link
Copy Markdown

Summary

  • when a collapsed task list exceeds maxVisible, normally anchor its window at the first unfinished task in the configured sort order
  • if fewer than maxVisible tasks remain after that focus point, backfill from earlier tasks so every oversized collapsed window still contains exactly maxVisible task rows
  • show separate … N earlier and … N later markers, with the final window used when every task is complete
  • follow Pi's global app.tools.expand state (Ctrl-O by default) without registering a competing shortcut or persisting temporary expansion state
  • retain showAll behavior and keep hiddenAt as a legacy-compatible setting

Testing

  • npx vitest run test/task-widget.test.ts (38 tests)
  • npm run lint
  • npm run typecheck
  • npm test (193 tests)
  • npm run build
  • independent review: approved

Notes

  • lists at or below maxVisible remain fully visible
  • all three commits are GPG-signed and remain independent of the firework animation patch
  • CHANGELOG.md and generated dist files are unchanged

@xz-dev
xz-dev force-pushed the feat/task-widget-focus-window branch from d23293b to 4f7b8d8 Compare July 25, 2026 06:54
xz-dev added a commit to xz-dev/pi-tasks that referenced this pull request Jul 26, 2026
@xz-dev
xz-dev force-pushed the feat/task-widget-focus-window branch 2 times, most recently from 17ff0cd to da9ccfe Compare July 26, 2026 15:54
@xz-dev
xz-dev force-pushed the feat/task-widget-focus-window branch from da9ccfe to 6c45ca9 Compare July 26, 2026 16:04
xz-dev added a commit to xz-dev/pi-tasks that referenced this pull request Jul 26, 2026
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.

1 participant