Skip to content

shell-updates#440

Open
pr3khar wants to merge 8 commits into
mainfrom
pr3-shell-updates
Open

shell-updates#440
pr3khar wants to merge 8 commits into
mainfrom
pr3-shell-updates

Conversation

@pr3khar
Copy link
Copy Markdown
Contributor

@pr3khar pr3khar commented Apr 8, 2026

Shell parity and layout (Shell, Bottom tray, Copilot Shell)
Aligned behavior and visuals across standalone Shell, BottomTray, and CopilotShell: containers, headers, threads, thread lists, welcome screens, conversation starters, and composers.

CopilotShell: added a ThreadListContainer plus threadList.scss so its thread list matches the other layouts more closely.

API and state:
SharedChatUIProps / OpenUIChat: new optional showAssistantLogo; composed layouts pass it through.

useScrollToBottom: respects scroll-padding-top when scrolling to the last user message, and clamps scroll position with Math.max(..., 0) to avoid odd negative scrolls.

@pr3khar pr3khar requested a review from ankit-thesys April 8, 2026 22:05
pr3khar and others added 6 commits April 10, 2026 15:20
- Sidebar: stop click propagation on toggle button and collapsed
  NewChatButton so toggle/new-chat intent always wins over the
  container's click-to-expand handler
- Sidebar: collapse the two isMobile effects into one, using a
  previousIsMobileRef to detect viewport transitions and avoid the
  stale-closure race between the open-state setter and animation
  state machine
- useComposerState: emit a one-shot dev-mode warning when the hook
  is used outside a ComposerStateProvider (silent fallback to local
  state otherwise breaks cross-component features like isDrafting)
- ThreadListContainer (BottomTray, CopilotShell): apply
  portalThemeClassName to portaled DropdownMenu.Content so non-default
  themes propagate into the dropdown subtrees
- ConversationStarter (Shell, BottomTray, CopilotShell): type
  isValidElement<{ children?: ReactNode }>(icon) so icon.props.children
  is accessible under React 19's stricter ReactElement typing

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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