-
|
Hi, first of all, thanks for all that you do! I first noticed this on my app, then later noted the same in your demo as well. Steps
Expected resultPanel increases the size smoothly from its minimum size. Actual resultPanel snaps from the minimum size to a larger size. Is this the expected behavior? If yes, I'm confused about the values they snap to when increasing the size. Thanks in advance. |
Beta Was this translation helpful? Give feedback.
Replies: 4 comments 5 replies
-
|
This is version 4.5.2: firefox-4-5-2.movAnd this is the latest version: firefox-latest.movIt's a bit subtle but I agree, the resize behavior does look jumpier than expected. |
Beta Was this translation helpful? Give feedback.
-
|
Thanks for the report, @Chevindu. I believe this issue has been fixed in #644 and released in react-resizable-panels@4.5.6 Let me know if you see anything else ❤️ → ☕ givebrian.coffee |
Beta Was this translation helpful? Give feedback.
-
|
Update to this issue: The following unit test (added to test("edge case discussions/643", () => {
(
[
[4, l([10, 90])],
// This step will fail without the collapsible change that caused this regression
[6, l([20, 80])],
// These steps will fail with the previous collapsible change (this is the regression/bug)
[10, l([20, 80])],
[15, l([25, 75])],
[25, l([35, 65])],
[40, l([50, 50])],
[50, l([50, 50])]
] satisfies [number, Layout][]
).forEach(([delta, expectedLayout]) => {
expect(
adjustLayoutByDelta({
delta,
initialLayout: l([10, 90]),
panelConstraints: c([
{
collapsedSize: 10,
collapsible: true,
defaultSize: 10,
maxSize: 50,
minSize: 20
},
{}
]),
prevLayout: l([10, 90]),
trigger: "mouse-or-touch"
})
).toEqual(expectedLayout);
});
}); |
Beta Was this translation helpful? Give feedback.
-
|
No worries! Thanks for letting me know 🙂
…On Mon, Feb 2, 2026 at 5:22 AM Chevindu Wickramathilaka < ***@***.***> wrote:
Hi, I tried version 4.5.8 this morning and can confirm the jumpy behavior
I observed is fixed. The demos also work smoothly on Firefox now.
I'm sorry about the delayed response, and thanks for looking into this
@bvaughn <https://github.com/bvaughn>!
—
Reply to this email directly, view it on GitHub
<#643 (reply in thread)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAHHHJEBA6Z5NYZSQHZXS34J3NINAVCNFSM6AAAAACTNAWXXOVHI2DSMVQWIX3LMV43URDJONRXK43TNFXW4Q3PNVWWK3TUHMYTKNRWHA3TONY>
.
You are receiving this because you were mentioned.Message ID:
<bvaughn/react-resizable-panels/repo-discussions/643/comments/15668777@
github.com>
|
Beta Was this translation helpful? Give feedback.
The original collapse behavior has been re-enabled, but with a fix for this issue, in version 4.5.7
Please let me know if you see anything else unexpected, @Chevindu
❤️ → ☕ givebrian.coffee