feat/pane-expansion : Add expansion anchors and enforce 250dp minimum…#81
Conversation
There was a problem hiding this comment.
Code Review
This pull request configures pane expansion anchors for the list-detail scaffold, applies a custom layout modifier to enforce a minimum width on both the list and detail panes, and adjusts z-indices and elevations for proper layering. The reviewer suggests extracting the duplicated layout logic into a reusable Modifier extension function that handles clipping and collapsed states to prevent visual bleeding. Additionally, the reviewer highlights a discrepancy between the defined snapping anchors and the initial pane proportion of 0.3f, which could cause a jumpy user experience.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
f01dc21 to
c6281c2
Compare
cka-dev
left a comment
There was a problem hiding this comment.
This is a nice update.
Only one thing: can you run the screenshot tests and make sure they are passing.
I think the change is affecting some of the existing screenshot tests.
… width for adaptive layout
c6281c2 to
6e13e7e
Compare
Improves the adaptive Master-Detail layout (CahierNavigationSuite) resizing behavior on large-screen devices. It adds snap-to anchors and guarantees that neither the list nor the detail pane is squished past legibility during resizing.
Key Changes