Skip to content

Expand top-to-bottom split resize target - #954

Open
brsbl wants to merge 1 commit into
bb/automation-breadcrumb-pane-controls-thr_cxz5rwzns3from
bb/top-bottom-split-resize-target-thr_cxz5rwzns3
Open

Expand top-to-bottom split resize target#954
brsbl wants to merge 1 commit into
bb/automation-breadcrumb-pane-controls-thr_cxz5rwzns3from
bb/top-bottom-split-resize-target-thr_cxz5rwzns3

Conversation

@brsbl

@brsbl brsbl commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • make split divider hit regions explicit and symmetric across both orientations
  • preserve the one-pixel visual seam and existing resize persistence behavior
  • add focused top-to-bottom drag regression coverage

Verification

  • pnpm exec turbo run test --filter=@bb/app -- --run src/views/thread-detail/SplitThreadArea.test.tsx (33 passed)
  • pnpm exec turbo run typecheck --filter=@bb/app

@brsbl
brsbl force-pushed the bb/top-bottom-split-resize-target-thr_cxz5rwzns3 branch 4 times, most recently from 3fcfa83 to a6463b8 Compare August 3, 2026 21:46
@brsbl

brsbl commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator Author

Final stack update:

  • Rebased onto PR Add automation breadcrumbs and pane full-screen controls #953 at 76d9d32; top head is a6463b8.
  • Top-to-bottom split dividers retain the one-pixel visual seam and now expose the same effective 12px drag target as left-to-right splits.
  • Focused top-stack pane suite: 46 tests passed, including both resize orientations and the aligned pane-header controls.
  • Full combined package matrix passed locally; BB desktop QA confirmed both orientations resize easily.
  • No unresolved review threads.

Not merged.

@brsbl
brsbl force-pushed the bb/top-bottom-split-resize-target-thr_cxz5rwzns3 branch 3 times, most recently from dadb411 to b9dbe0a Compare August 4, 2026 01:52
@brsbl
brsbl force-pushed the bb/top-bottom-split-resize-target-thr_cxz5rwzns3 branch from b9dbe0a to 4452051 Compare August 4, 2026 02:02
@brsbl
brsbl force-pushed the bb/top-bottom-split-resize-target-thr_cxz5rwzns3 branch from 4452051 to 21b585f Compare August 4, 2026 02:12
@brsbl
brsbl force-pushed the bb/top-bottom-split-resize-target-thr_cxz5rwzns3 branch from 21b585f to 758a21d Compare August 4, 2026 02:52
@brsbl
brsbl marked this pull request as ready for review August 4, 2026 04:44
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.

1 participant