feat: Add keyboard shortcut to scroll info panels to top

[mtrezza]

Pull Request

• Report security issues confidentially.
• Any contribution is under this license.

Feature

Add keyboard shortcut to scroll info panels to top.

Summary by CodeRabbit

New Features

• Added a keyboard shortcut to quickly scroll data browser panels to the top (default: U key). When triggered, the shortcut scrolls all panel columns to the top and pauses any active auto-scroll functionality. Users can fully customize the shortcut key and modifiers through keyboard shortcuts preferences.

[parse-github-assistant]

🚀 Thanks for opening this pull request!

[coderabbitai]

📝 Walkthrough

Walkthrough

This PR adds a new keyboard shortcut dataBrowserScrollInfoPanelsToTop (key 'u') to the DataBrowser. The shortcut scrolls the aggregation panel and panel columns to the top, pauses auto-scroll if active, and prevents default behavior. Implementation spans keyboard event handling, settings UI configuration, and default shortcut preferences.

Changes

Cohort / File(s)	Summary
New Keyboard Shortcut: Scroll Info Panels to Top src/dashboard/Data/Browser/DataBrowser.react.js, src/dashboard/Settings/KeyboardShortcutsSettings.react.js, src/lib/KeyboardShortcutsPreferences.js	Introduces dataBrowserScrollInfoPanelsToTop shortcut with handler logic that scrolls aggregation and panel column refs to top and pauses auto-scroll. Adds configuration UI field and default mapping (key: 'u', all modifiers false). Reorders existing "Run script on selected rows" handler to execute after the new shortcut check.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• feat: Add auto-scrolling for info panels #3149: New keyboard shortcut directly manipulates aggregationPanelRef and panelColumnRefs while pausing auto-scroll state, overlapping with existing auto-scroll methods in DataBrowser.
• fix: Filter text field in data browser partly looses focus when hitting enter key to apply filter #2992: Modifies the same DataBrowser.handleKey function with keyboard event guard logic; this PR adds a new early-return path for the scroll-to-top shortcut in the same handler.
• feat: Add keyboard shortcuts for quick actions in data browser #3073: Modifies the same keyboard shortcut plumbing across DataBrowser, KeyboardShortcutsSettings, and KeyboardShortcutsPreferences files.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description is incomplete relative to the template. It includes a brief feature statement but is missing the required 'Issue' and 'Approach' sections, and does not address the 'Tasks' checklist.	Add the 'Issue' section to describe or link the issue being closed, add the 'Approach' section detailing the implementation changes, and address the 'Tasks' checklist regarding tests and documentation.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title directly and clearly summarizes the main change: adding a keyboard shortcut to scroll info panels to top, matching the core functionality implemented across all three modified files.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

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

---

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments

src/dashboard/Data/Browser/DataBrowser.react.js (1)

833-833: shortcuts variable is now declared twice in handleKey.

Line 833 declares const shortcuts = this.state.keyboardShortcuts; at the outer function scope, making the identical declaration at line 1112 (inside the default block) redundant. Consider removing the inner one and reusing the outer variable.

♻️ Remove redundant declaration

       default: {
         // Handle custom keyboard shortcuts from server
-        const shortcuts = this.state.keyboardShortcuts;
         if (!shortcuts) {
           break;
         }

Also applies to: 1112-1112

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.

---

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.5.0-alpha.5

[parseplatformorg]

🎉 This change has been released in version 8.5.0