Skip to content

Conversation

@mtrezza
Copy link
Member

@mtrezza mtrezza commented Nov 30, 2025

New Pull Request Checklist

Feature

Remember info panel width and count across browser sessions. Currently, this is lost when refreshing the browser page.

Approach

Remember info panel width and count across browser sessions in local storage.

Summary by CodeRabbit

  • New Features
    • Data Browser now remembers aggregation panel configuration, including panel width and count, across sessions. Panel layout adjustments are automatically saved when resizing or adding/removing panels.

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

@parse-github-assistant
Copy link

🚀 Thanks for opening this pull request! We appreciate your effort in improving the project. Please let us know once your pull request is ready for review.

@coderabbitai
Copy link

coderabbitai bot commented Nov 30, 2025

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

This PR adds localStorage persistence for aggregation panel width and count state in the DataBrowser component. Initial values are read from localStorage with defaults (300px width, 1 count), and these values are persisted on resize completion, panel addition/removal, and batch calculation completion. Panel width derivations are recalculated when panels change.

Changes

Cohort / File(s) Change Summary
Aggregation panel state persistence
src/dashboard/Data/Browser/DataBrowser.react.js
Added localStorage keys and read/persist logic for panel width and count; read initial values from localStorage (defaults: 300, 1) in constructor; persist to localStorage on resize stop, panel add/remove, and batch calculations; adjust derived panel width calculations when panels are added or removed

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Verify localStorage key definitions and default values (300 for width, 1 for count)
  • Confirm all user actions that modify panelWidth or panelCount are covered by persistence calls
  • Check that parsed values from localStorage are properly validated and fallback correctly
  • Review panel width recalculation logic when panels are added/removed to ensure maxWidth constraints are respected

Possibly related PRs

✨ 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 3f28699 and 522f4a2.

📒 Files selected for processing (1)
  • src/dashboard/Data/Browser/DataBrowser.react.js (7 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

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

@parseplatformorg
Copy link
Contributor

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.

@mtrezza mtrezza merged commit d188030 into parse-community:alpha Nov 30, 2025
10 of 11 checks passed
@mtrezza mtrezza deleted the feat/remember-panel-width branch November 30, 2025 18:59
parseplatformorg pushed a commit that referenced this pull request Nov 30, 2025
# [8.1.0-alpha.10](8.1.0-alpha.9...8.1.0-alpha.10) (2025-11-30)

### Features

* Remember info panel width and count across browser sessions ([#3031](#3031)) ([d188030](d188030))
@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 8.1.0-alpha.10

@parseplatformorg parseplatformorg added the state:released-alpha Released as alpha version label Nov 30, 2025
parseplatformorg pushed a commit that referenced this pull request Dec 1, 2025
# [8.1.0](8.0.0...8.1.0) (2025-12-01)

### Bug Fixes

* Data browser export triggers confirmation dialog for navigating with selected rows ([#3037](#3037)) ([c6334cd](c6334cd))
* Info panel data not reloading when clicking refresh button in data browser ([#3027](#3027)) ([8f91d15](8f91d15))
* Info panel may show stale data of previous objects when refreshing the data table or navigating between classes ([#3036](#3036)) ([2565f0e](2565f0e))
* No individual data loading indicators when displaying multiple info panels ([#3026](#3026)) ([5ca8218](5ca8218))
* Selected saved filter is not expanded in sidebar when reloading browser page ([#3029](#3029)) ([2e7850e](2e7850e))
* Sync-scrolling multiple panels jumps to bottom of shortest panel when scrolling up ([#3023](#3023)) ([3f85f89](3f85f89))
* Sync-scrolling multiple panels stops at bottom of shortest panel when scrolling down ([#3024](#3024)) ([bf46938](bf46938))

### Features

* Add `cookieSessionStore` option to support multi-replica deployments ([#3016](#3016)) ([3eb4b05](3eb4b05))
* Add displaying multiple info panels and batch navigation ([#3020](#3020)) ([7ac3186](7ac3186))
* Automatically resize info panel sidebar width when adding or removing panels ([#3025](#3025)) ([1032301](1032301))
* Remember info panel width and count across browser sessions ([#3031](#3031)) ([d188030](d188030))
* Remember selected column when navigating between saved filters of a class with auto-load first row enabled ([#3028](#3028)) ([964584c](964584c))
* Upgrade to parse 7.1.2 ([#3038](#3038)) ([c6c1787](c6c1787))
@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 8.1.0

@parseplatformorg parseplatformorg added the state:released Released as stable version label Dec 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

state:released Released as stable version state:released-alpha Released as alpha version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants