Skip to content

fix: enable independent scroll on kanban board columns#114

Merged
jcanizalez merged 2 commits intomainfrom
fix/kanban-column-scroll-v2
Mar 26, 2026
Merged

fix: enable independent scroll on kanban board columns#114
jcanizalez merged 2 commits intomainfrom
fix/kanban-column-scroll-v2

Conversation

@jcanizalez
Copy link
Copy Markdown
Owner

Summary

  • App.tsx wrapper was not a flex container, so TaskBoardView's flex-1 had no effect and columns grew unbounded
  • TaskBoardView's content div used overflow-auto in kanban mode, causing the entire board to scroll as one block instead of per-column
  • Made App.tsx wrapper a flex column with min-h-0 so height constraints propagate
  • Switched kanban content div to overflow-hidden so each column scrolls independently

Closes #90

The parent wrapper in App.tsx was not a flex container, so
TaskBoardView's flex-1 had no effect and columns grew unbounded.
Additionally, TaskBoardView's content div used overflow-auto in
kanban mode, causing the entire board to scroll as one block.

- Make App.tsx wrapper a flex column so height constraints propagate
- Switch kanban content div to overflow-hidden so each column
  scrolls independently via its own overflow-y-auto
Copilot AI review requested due to automatic review settings March 26, 2026 15:26
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes Kanban board scrolling behavior by ensuring height constraints propagate through the renderer layout so each column can scroll independently (closes #90).

Changes:

  • Make the main content wrapper in App.tsx a flex column with min-h-0 so nested flex-1/overflow constraints work.
  • In TaskBoardView, switch the kanban-mode content container from overflow-auto to overflow-hidden (and make it a flex container) so per-column overflow-y-auto scrolling takes effect.
  • Bump root package.json version to 0.4.0-beta.0.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
src/renderer/components/TaskBoardView.tsx Adjusts the content container layout/overflow to enable independent kanban column scrolling.
src/renderer/App.tsx Makes the immediate parent layout a flex column with min-h-0 so flex-1 height constraints propagate to the task board.
package.json Updates the application version (not described in the PR summary).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

{
"name": "vibegrid",
"version": "0.3.1",
"version": "0.4.0-beta.0",
Copy link

Copilot AI Mar 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR description is focused on fixing Kanban scrolling/layout, but this also bumps the root app version from 0.3.1 to 0.4.0-beta.0. Since the release workflow and the app’s About/version surfaces read from package.json, this version change can affect release artifacts/update metadata; either justify it in the PR description or keep the version unchanged for this fix-only PR.

Copilot uses AI. Check for mistakes.
@jcanizalez jcanizalez merged commit e0867bc into main Mar 26, 2026
5 checks passed
@jcanizalez jcanizalez deleted the fix/kanban-column-scroll-v2 branch March 26, 2026 15:36
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.

Tasks board view: no scroll when too many cards

2 participants