Skip to content

Fixed an issue where ALT+F5 for execute query in Query Tool shows cro…#9763

Open
anilsahoo20 wants to merge 1 commit intopgadmin-org:masterfrom
anilsahoo20:GH-9570
Open

Fixed an issue where ALT+F5 for execute query in Query Tool shows cro…#9763
anilsahoo20 wants to merge 1 commit intopgadmin-org:masterfrom
anilsahoo20:GH-9570

Conversation

@anilsahoo20
Copy link
Contributor

@anilsahoo20 anilsahoo20 commented Mar 18, 2026

…sshairCursor icon for rectangularSelection. #9570

Summary by CodeRabbit

  • Bug Fixes
    • Updated keyboard shortcut for rectangular selection in the code editor to Alt+Ctrl combination
    • Modified activation trigger for crosshair cursor in the code editor to Control key

@coderabbitai
Copy link

coderabbitai bot commented Mar 18, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: bc825537-ac72-4fc0-88c5-179a3c207901

📥 Commits

Reviewing files that changed from the base of the PR and between 2576548 and f0abe2b.

📒 Files selected for processing (1)
  • web/pgadmin/static/js/components/ReactCodeMirror/components/Editor.jsx

Walkthrough

The Editor.jsx ReactCodeMirror component is updated to configure two CodeMirror extensions with specific event filters and key bindings: rectangular selection now requires Alt+Ctrl key combination, and crosshair cursor behavior is bound to the Control key.

Changes

Cohort / File(s) Summary
ReactCodeMirror Extension Configuration
web/pgadmin/static/js/components/ReactCodeMirror/components/Editor.jsx
Modified rectangularSelection() extension to require Alt+Ctrl key combination via eventFilter, and updated crosshairCursor() extension to bind to Control key via key parameter.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: fixing an issue where ALT+F5 for executing queries in the Query Tool incorrectly displays a crosshairCursor icon for rectangular selection.
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 unit tests (beta)
  • Create PR with unit tests
📝 Coding Plan
  • Generate coding plan for human review comments

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.

Tip

CodeRabbit can generate a title for your PR based on the changes.

Add @coderabbitai placeholder anywhere in the title of your PR and CodeRabbit will replace it with a title based on the changes in the PR. You can change the placeholder by changing the reviews.auto_title_placeholder setting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant