fix: Info panel auto-scrolling not pausing during UI interaction like text selection or context menu display

[mtrezza]

Pull Request

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

Summary by CodeRabbit

• New Features

  • Auto-scroll can be recorded using the Command/Meta key; wheel scrolls while held start a recording and accumulate deltas.
  • Left-mouse button interactions now pause/resume auto-scroll more reliably.

• Bug Fixes

  • Improved native context-menu detection and lifecycle handling.
  • More robust auto-scroll state management with smoother resume after interruptions and better handling of simultaneous inputs.

[parse-github-assistant]

🚀 Thanks for opening this pull request!

[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.

[coderabbitai]

📝 Walkthrough

Walkthrough

Tracks Command/Meta key, left mouse button, and native context-menu state in DataBrowser to pause/resume and record user-driven wheel input for auto-scroll control; adds helpers to detect native context-menu lifecycle and updates auto-scroll start/stop/stepping and recording logic accordingly. (46 words)

Changes

Cohort / File(s)	Summary
DataBrowser core src/dashboard/Data/Browser/DataBrowser.react.js	Adds commandKeyPressed state; instance fields mouseButtonPressed, nativeContextMenuTracker; new handlers handleMouseButtonDown, handleMouseButtonUp; binds updated; initializes/restores new fields.
Auto-scroll control & recording src/dashboard/Data/Browser/DataBrowser.react.js	Reworks auto-scroll key and wheel handling: enter/exit recording on first wheel with Command held, accumulate recordedScrollDelta, commit recording on key release, reset commandKeyPressed when auto-scroll stops, and add temporary blocked/unblock wait before resuming stepping.
Native context-menu detection src/dashboard/Data/Browser/DataBrowser.react.js	Replaces inline event wiring with setupNativeContextMenuDetection() that attaches pointer/keyboard/visibility/blur handlers and returns a disposer; integrates lifecycle into init/cleanup.
Auto-scroll blocking logic src/dashboard/Data/Browser/DataBrowser.react.js	Extends isAutoScrollBlocked to consider mouseButtonPressed and commandKeyPressed; adds left-button listeners to pause/resume auto-scroll and coordinates blocking state with recording and stepping logic.

Sequence Diagram

sequenceDiagram
    actor User
    participant DataBrowser
    participant AutoScroll
    participant NativeOS as Native OS / ContextMenu

    User->>DataBrowser: Press Command (Meta)
    DataBrowser->>DataBrowser: commandKeyPressed = true

    User->>DataBrowser: Wheel scroll
    DataBrowser->>AutoScroll: stop ongoing auto-scroll
    DataBrowser->>DataBrowser: enter recording (isRecordingAutoScroll = true)
    DataBrowser->>DataBrowser: accumulate recordedScrollDelta

    User->>NativeOS: Open native context menu (optional)
    NativeOS->>DataBrowser: nativeContextMenuTracker triggers pause
    DataBrowser->>DataBrowser: mouseButtonPressed / native menu block set

    User->>DataBrowser: Release Command
    DataBrowser->>DataBrowser: handleAutoScrollKeyUp -> commit recording if delta
    DataBrowser->>AutoScroll: resume/step after unblock (with optional delay)

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• feat: Various improvements for auto-scrolling info panel #3151 — Related changes to DataBrowser auto-scroll behavior and context-menu detection/pausing logic.
• feat: Allow to pause auto-scroll of info panel by holding the Option key #3160 — Similar modifier-key driven auto-scroll pause handling (Option/Alt-focused).
• feat: Add auto-scrolling for info panels #3149 — Modifications to auto-scroll state initialization and key/wheel handlers in the same component.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The PR description is largely incomplete, missing the Issue link, detailed Approach description, and Tasks checklist from the required template.	Complete the PR description by filling in the Issue link, providing a detailed Approach section explaining the changes, and checking/unchecking the Tasks checklist as applicable.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The PR title clearly summarizes the main change: adding pause functionality to auto-scrolling during UI interactions like text selection and context menu interaction.
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

---

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]

🎉 This change has been released in version 8.3.0-alpha.39