fix: Info panel covers whole sidebar if fewer objects than panels in multi-panel scenario

[mtrezza]

Summary by CodeRabbit

• Improvements
  • Panel toggle buttons now correctly display the count of panels (e.g., "Hide 2 Panels" vs. "Hide Panel").
  • Panel width is now properly calculated and displayed when panels are hidden.
  • Empty panel states now display aggregation data instead of blank space.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[parse-github-assistant]

🚀 Thanks for opening this pull request!

[coderabbitai]

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Updates panel toggle button labels to dynamically pluralize based on panel count. Introduces effectivePanelWidth calculation in DataBrowser to account for hidden panels, persists width adjustments to localStorage, and modifies panel rendering logic to display aggregation panels when no panels are currently shown.

Changes

Cohort / File(s)	Change Summary
Toolbar label pluralization src/components/Toolbar/Toolbar.react.js	Dynamically pluralizes panel toggle button labels: "Hide Panel" becomes "Hide {panelCount} Panels" and "Show Panel" becomes "Show {panelCount} Panels" when panelCount > 1.
DataBrowser panel width and rendering src/dashboard/Data/Browser/DataBrowser.react.js	Introduces effectivePanelWidth calculation to adjust rendered width based on hidden panels; translates effective width back to full panel width on resize and persists to localStorage. Updates all UI components (BrowserTable, ResizableBox, per-panel rendering) to use effectivePanelWidth. Renders single AggregationPanel when no panels displayed, pulling data from cached multiPanelData or AggregationPanelData; adjusts isLoadingCloudFunction and selectedObjectId propagation accordingly.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

• DataBrowser.react.js: Multiple interrelated logic changes affecting width calculations, state propagation, and render paths require careful verification of edge cases (hidden panels, no panels displayed, resize behavior) and localStorage persistence.
• Toolbar.react.js: Label pluralization logic is straightforward but should be verified for correct panelCount handling across all toggle states.

Possibly related PRs

• feat: Add displaying multiple info panels and batch navigation #3020: Modifies both Toolbar.react.js and DataBrowser.react.js to implement multi-panel behavior with panelCount, toggle labels, and width/rendering handling.
• feat: Remember info panel width and count across browser sessions #3031: Modifies DataBrowser.react.js panel sizing logic including effectivePanelWidth calculation and panelCount/panelWidth persistence to localStorage.
• feat: Automatically resize info panel sidebar width when adding or removing panels #3025: Modifies panel width handling in DataBrowser.react.js to recalculate effectivePanelWidth when hidden panels exist.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6aadff6 and 004e764.

📒 Files selected for processing (2)

• src/components/Toolbar/Toolbar.react.js (1 hunks)
• src/dashboard/Data/Browser/DataBrowser.react.js (5 hunks)

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[parseplatformorg]

✅ Snyk checks have passed. No issues have been found so far.

Status	Scanner	Critical	High	Medium	Low	Total (0)
✅	Open Source Security	0	0	0	0	0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

[parseplatformorg]

🎉 This change has been released in version 8.2.0-alpha.2