Skip to content

fix(sidebar): refine session rows and todo disclosure - #1734

Merged
vastsa merged 4 commits into
mainfrom
feat/sidebar-session-date-action-swap
Oct 17, 2026
Merged

vastsa merged 4 commits into
mainfrom
feat/sidebar-session-date-action-swap

Conversation

@vastsa

@vastsa vastsa commented Oct 17, 2026

Copy link
Copy Markdown
Owner

Summary

  • Keep session date and menu actions in one shared slot, remove the divider, and show older dates as month/day.
  • Give the TodoDock disclosure a 36px hit target and size its arrow independently at 18px.
  • Seed the existing Session time preference in sidebar coverage.

Validation

  • pnpm build:js
  • pnpm build:host
  • pnpm --filter @pi-desktop/desktop typecheck
  • pnpm lint
  • Targeted sidebar and TodoDock renderer tests
  • pnpm test:e2e:todos
  • pnpm test:e2e:layout (201/201)

Keep each project session date and overflow action in one stable trailing slot so hover and keyboard focus switch between them without shifting the row. Use month and day for older activity dates to keep the sidebar metadata compact.
Use a 36px square disclosure target and size its arrow to 18px. The
shared button padding previously squeezed the larger icon, so give the
composed button selector its own zero-padding rule.

Seed the existing Session time preference in sidebar coverage so the
new row-date behavior is exercised under its enabled setting.
Copilot AI balanced review requested due to automatic review settings October 17, 2026 07:54

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Keep the translated UX references aligned with the new session-date
slot and the TodoDock disclosure dimensions. This also preserves the
English/Chinese table shape required by the docs locale gate.
Match keyboard and touch assertions to the shared session metadata slot.
The date/action layout moved out of the former row-actions wrapper, so
the tests should cover the current focus and no-hover selectors.
@vastsa
vastsa merged commit 21f5d11 into main Oct 17, 2026
7 checks passed
@vastsa
vastsa deleted the feat/sidebar-session-date-action-swap branch October 17, 2026 08:18

This branch was previously deployed

1 inactive deployment
Preview — ffb4a8a9 Deployed Oct 17, 2026 by vercel[bot]
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.

2 participants