mirror of
https://github.com/CopilotKit/CopilotKit.git
synced 2026-09-14 16:26:20 +08:00
Benjamin Taylor
a3c5f079ed
Children of `CopilotSidebar` cannot use `height: 100%`. Both wrappers the sidebar puts around your app -- `.copilotKitSidebarContentWrapper` and `.copilotKitModalChildrenWrapper` (which had no CSS rule at all) -- are auto-height blocks, so a percentage height on a child has no definite containing block and collapses to content height. Add an opt-in `fullHeightChildren` prop that gives the content wrapper a one-viewport height and lets the children wrapper fill it. It is opt-in because the content wrapper wraps the entire consumer app, and giving every react-ui sidebar user a flex column with a fixed height would reflow apps that never asked for it. The height is a viewport unit, not `100%`: `100%` only resolves when every ancestor (html/body/#root) also declares a height, which react-ui neither sets nor can guarantee, so it would silently no-op in a stock Next.js app. `min-height: 0` on the children wrapper clears the flex-item `min-height: auto` floor so tall content scrolls inside the child instead of stretching the wrapper past the viewport. Fixes #261 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Co-Authored-By: Ashish Shaw <77574570+ashish4143@users.noreply.github.com>