Show a focus ring on sidebar thread and draft rows

[ryanilano]

Sidebar thread and draft rows are focusable (role="button", tabIndex={0}) but set outline-none with no replacement, so keyboard focus is invisible. This adds the same inset focus-visible ring the tool-call region already uses.

Verify: press Tab into the sidebar; the focused row shows a ring. Mouse users see no change.

Screenshots to follow.

Summary by CodeRabbit

• Accessibility
  • Sidebar draft and thread rows now display a visible focus ring when reached by keyboard navigation.

[macroscopeapp]

Approvability

Verdict: Approved at 2978ef5

Macroscope's review found this PR approvable — This single-file change adds a keyboard-visible focus ring to two already focusable sidebar rows. It is a small, self-contained visual accessibility improvement that leaves mouse behavior and application workflows unchanged.

^{You can add or adjust custom eligibility rules. Learn more.}

[coderabbitai]

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository: pingdotgg/t3code/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 7280c73b-bbe0-4fe1-836a-4f482c718598

📥 Commits

Reviewing files that changed from the base of the PR and between e4eb997 and 2978ef5.

📒 Files selected for processing (1)

• apps/web/src/components/Sidebar.tsx

Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.

---

📝 Walkthrough

Walkthrough

Draft and thread sidebar rows now include focus-visible ring styles. The styles apply to draft rows and to both card and slim thread rows.

Changes

Sidebar focus visibility

Layer / File(s)	Summary
Add focus rings to sidebar rows apps/web/src/components/Sidebar.tsx	Draft rows and card and slim thread rows add focus-visible ring styles.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix

Suggested reviewers: juliusmarminge

Merge Risk: ⚪ Minimal · up to 2978e

The sidebar rows gain the intended keyboard focus indication; no material merge risk remains.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description explains the problem, the change, and the verification step. It does not use the required sections, provide the required UI screenshots, or include the checklist.	Add the What Changed, Why, UI Changes, and Checklist sections. Include before-and-after screenshots for the UI change, then complete the checklist.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly and concisely describes the main change: adding focus rings to sidebar thread and draft rows.

• Fix all pre-merge checks with AI

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create a new PR

---

_{Comment @coderabbitai help to get the list of available commands.}

[ryanilano]

After: Tab from the sidebar search box to a thread row. The row now shows a focus ring.