Skip to content

Fix sidebar scrolling when conversation list overflows screen height#12

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/fix-11
Draft

Fix sidebar scrolling when conversation list overflows screen height#12
Copilot wants to merge 2 commits intomainfrom
copilot/fix-11

Conversation

Copy link
Copy Markdown

Copilot AI commented Jun 4, 2025

The sidebar was not scrollable when the conversation list exceeded the available screen height, causing conversations to be cut off and inaccessible to users.

Problem

The sidebar container used h-full but lacked proper flex layout, preventing the thread list section from taking the remaining space and scrolling independently when content overflows.

Solution

Added flex flex-col classes to the main sidebar container to enable proper space distribution:

  • Header takes its natural height
  • Thread list (flex-1) takes remaining available height
  • overflow-y-auto can now work properly within the constrained height

Changes

  • Modified src/components/sidebar/Sidebar.tsx: Added flex flex-col to sidebar container (minimal 1-line change)
  • Added src/components/sidebar/Sidebar.test.tsx: Comprehensive tests to verify layout structure and scrolling behavior

Testing

  • ✅ 4/4 new tests pass
  • ✅ Build succeeds without errors
  • ✅ No regressions in existing functionality

The fix ensures users can scroll through all conversations when the list grows beyond the sidebar height.

Fixes #11.

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

  • workers.cloudflare.com
    • Triggering command: node /home/REDACTED/work/agents-starter/agents-starter/node_modules/.bin/vite dev (dns block)

If you need me to access, download, or install something from one of these locations, you can either:


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: gvelesandro <6748281+gvelesandro@users.noreply.github.com>
Copilot AI changed the title [WIP] side bar doesn't scroll when list overflows screen Fix sidebar scrolling when conversation list overflows screen height Jun 4, 2025
Copilot AI requested a review from gvelesandro June 4, 2025 00:35
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.

side bar doesn't scroll when list overflows screen

2 participants