fix(web): respect motion settings in the thread details panel#22434

ThreadDetailsCard tracks inline panel presence and applies animation settings. PopoverPopup accepts an optional duration and uses it to set its animation CSS variable.

Changes: Thread panel animation

Layer / File(s)	Summary
Configurable popover duration apps/web/src/components/ui/popover.tsx	PopoverPopup accepts an optional animation duration. When provided, it applies the duration through a CSS variable and preserves the existing style value.
Thread panel presence and animation apps/web/src/components/chat/ThreadDetailsCard.tsx	ThreadDetailsCard tracks inline presence by scoped thread key, applies entry and closed-state styles, keeps content rendered during exit, marks closed content inert, and passes the configured duration or zero to the popover.

Priority: External image Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix

Suggested reviewers: juliusmarminge

The motion-settings change remains low-risk to merge but should address the fixed positioner timing and add coverage for inline close and reopen behavior.

Check name	Status	Explanation
Title check	Passed	The title clearly and concisely describes the main change: applying motion settings to the web Thread Details panel.
Description check	Passed	The description follows the required structure and explains the problem, change, scope, approval basis, verification, limitations, and UI evidence. It does not include an explicit maintainer approval …
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.

Full details: Description check

Explanation

The description follows the required structure and explains the problem, change, scope, approval basis, verification, limitations, and UI evidence. It does not include an explicit maintainer approval comment for discussion #22330, but the focused bug-fix exception is clearly justified.

🧪 Generate unit tests (beta)

• Create a new PR

🏁 Script executed:

shell

1	#!/bin/bash
2	fd -a '^AGENTS\.md$|^testing\.md$|^ThreadDetailsCard\.tsx$'
3	if [ -f AGENTS.md ]; then cat AGENTS.md; fi
4	if [ -f docs/internals/testing.md ]; then cat docs/internals/testing.md; fi
5	for d in apps apps/web apps/web/src apps/web/src/components apps/web/src/components/chat docs docs/internals; do
6	if [ -f "$d/AGENTS.md" ]; then printf '\n--- %s ---\n' "$d/AGENTS.md"; cat "$d/AGENTS.md"; fi
7	done
8	ast-grep outline apps/web/src/components/chat/ThreadDetailsCard.tsx
9	sed -n '1,240p' apps/web/src/components/chat/ThreadDetailsCard.tsx

Length of output: 38891

---

1	Learnt from: CR
2	Repo: pingdotgg/t3code PR: 0
3	File: AGENTS.md:0-0
4	Timestamp: 2026-10-16T06:14:31.198Z
5	Learning: Source excerpt:
6	# T3 Code
7
8	## A note from Theo
9
10	Channel both "measure twice, cut once" and "yagni". Fight scope creep. Try to honor the dev's intent in both a minimal and realistic fashion.

1	Learnt from: t3dotgg
2	Repo: pingdotgg/t3code PR: 22898
3	File: apps/web/src/components/chat/composerFocusRedirect.ts:56-59
4	Timestamp: 2026-10-21T13:42:36.510Z
5	Learning: In apps/web/src/components/ChatView.tsx, ChatRightPanel mounts only while rightPanelPresent is true and activeThreadRef exists. apps/web/src/panelAnimations.ts usePanelPresence ends panel presence after the close-animation duration, or immediately when animations are disabled. RightPanelSheet's keepMounted does not preserve the launcher beyond this parent unmount. In apps/web/src/components/RightPanelTabs.tsx, RightPanelEmptyState keeps its window capture keydown listener registered until unmount, so data-surface-launcher-keys remains aligned with that listener during closing.