feat(web): indicate session activity in date picker - #1103
Merged
tiann merged 2 commits intoJul 24, 2026
Conversation
There was a problem hiding this comment.
Findings
- [Minor] Activity signal is color-only — the new
hasSessionActivitystate only changestext-[var(--app-fg)]vstext-[var(--app-hint)], while the day button label remains just the date. Screen-reader and forced-colors users get no session-activity indication, so the new feature is unavailable for them. Evidenceweb/src/components/SessionList.tsx:642
Suggested fix:Add matchingconst dateLabel = date.toLocaleDateString() const buttonLabel = hasSessionActivity ? t('sessions.timeFilter.dayWithActivity', { date: dateLabel }) : dateLabel <button aria-label={buttonLabel} title={buttonLabel}>
sessions.timeFilter.dayWithActivityentries inweb/src/lib/locales/en.tsandweb/src/lib/locales/zh-CN.ts.
Summary
- Review mode: initial
- One minor accessibility gap found in the new date activity indicator. Residual risk: visual-only behavior around color themes was not exercised here.
Testing
- Not run (automation; review only, no PR code execution).
HAPI Bot
There was a problem hiding this comment.
Findings
No findings.
Questions
None.
Summary
- Review mode: follow-up after new commits
- No issues found in the latest full PR diff. The previous bot accessibility finding is addressed at the current head. Residual risk: review-only pass; tests/build were not rerun here.
Testing
- Not run (automation; review only).
HAPI Bot
heavygee
added a commit
to heavygee/hapi
that referenced
this pull request
Jul 25, 2026
Manifest: dropped session-copy-link (tiann#1144) + acp-remote-resume (tiann#991/tiann#1128, merged upstream); re-parked garden-route (stale, 128 commits behind — owner re-thin) and cursor-model-error-bridge (stale re-thin base — owner re-thin); overseer summary-emit + summary-fallback retained. 25 live layers. Heal 56: three v0.23.4 integration gaps — SessionListSearch.sessionActivityDates required (tiann#1103) on /share picker, SessionLogPanel title plumbing (trace-graph merge drift), and ACP tiann#1028 title tests awaiting the inline-image sessionUpdateQueue. Rematerialized green @ driver 48703cdea (typecheck + vitest + web-dist verify). Co-authored-by: Cursor <cursoragent@cursor.com>
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.
Summary
Makes the session date-range picker show which dates contain session activity.
Follow-up to #1083.
Motivation
The date filter added in #1083 helps narrow a large session history, but every day currently has the same visual weight. Users still need to guess which dates contain sessions. Showing activity availability directly in the calendar makes useful dates visible at a glance without adding controls or preventing broader range selection.
Test plan
bun run test:webbun typecheckbun run build:webAI disclosure
Implementation and tests were assisted by OpenAI Codex (GPT-5.6).