[BUG] Division by zero panic in ResizablePanelGroup

[rajpreetcodes]

Bug Report

Platform

All platforms (Linux, macOS, Windows, WSL2)

Current Behavior

In �pps/desktop/src/components/resizable.rs:71-79, the drag fraction calculation divides by bounds width/height without checking for zero:

ust let fraction = match self.orientation { Orientation::Horizontal => { (event.event.position.x - bounds.left()) / (bounds.right() - bounds.left()) // Can be 0 } Orientation::Vertical => { (event.event.position.y - bounds.top()) / (bounds.bottom() - bounds.top()) // Can be 0 } };

If the panel group has zero width or height (e.g., during window initialization, minimized state, or layout edge cases), this causes a division by zero panic.

Expected Behavior

The calculation should guard against zero-size bounds and return a safe default fraction.

Steps To Reproduce

1. Run the desktop app
2. Resize window to minimum size or trigger a layout where panel bounds collapse to zero
3. Attempt to drag the resizable handle

Recurrence Probability

Sometimes - depends on window state and layout timing

Additional Context

Critical bug - causes immediate panic/crash. Should be fixed before any PRs are accepted.

[killer1panda]

I've investigated this issue, verified the root cause, and put together a fix with unit tests on my fork:

• Branch: fix/resizable-panel-zero-bound-panic
• Commit: 821aabf

Root Cause

When the resizable panel group collapses or has zero width/height during window initialization or layout edge cases, (offset) / (bounds.right() - bounds.left()) results in 0.0 / 0.0 = NaN or +/-inf. Passing this to self.clamp_fraction(fraction) triggers f32::clamp's internal assertion !self.is_nan(), causing an immediate panic crash.

Changes

1. Guard against non-positive span (span <= px(0.0)) in ResizablePanelGroup::resize, early-returning without modifying fraction or notifying listeners.
2. Ensure fraction.is_finite() before updating self.fraction.
3. Harden clamp_fraction and minimum_fraction with .is_finite() checks so non-finite floats safely resolve to boundaries instead of panicking.
4. Added unit test panel_fraction_handles_nan_and_infinities covering NaN, +/-inf, and invalid minimums.

All existing and new tests pass cleanly (cargo test --manifest-path apps/desktop/Cargo.toml).

Given that direct PRs are currently closed during the rewrite, happy to open a PR or let maintainers cherry-pick this commit whenever appropriate!