[Feature]: Run the Script pane with Ctrl+Enter (⌘+Enter on macOS)
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 83/100
- Issue type
- Feature
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- rust
- Domain
- documentation, frontend, testing
Research direction
Start with sqlbox.rs to follow the existing Ctrl+Enter handling, then find the Script pane's Run button and script editor. Keep the shortcut limited to editor focus and share its execution path with the button; preserve Enter and suggestion-list behavior. Update the tooltip and docs/user-guide/analysis.md, then add the described test verifying the script runs without changing its text.
Written by the indexing model from the issue text.
Description
Before you start
- I searched the existing issues and the roadmap and did not find a duplicate request.
What problem does this solve?
The Script pane (Analyze › Script) can only be run with its Run button. While writing and testing a script, every run means leaving the keyboard for the mouse, and then clicking back into the box to keep editing.
The SQL pane already runs its query with Ctrl+Enter (sqlbox.rs, documented in docs/user-guide/data.md), so the Script pane is the odd one out.
Proposed solution
With the focus in the Script box, Ctrl+Enter (⌘+Enter on macOS, egui::Modifiers::COMMAND) runs the script, as the Run button does:
- The key is taken before the box sees it, so it runs the script and does not break the line. This is how
sqlbox.rsdoes it. - It only applies while the Script box has the focus. Elsewhere the key does nothing new.
- The box keeps the focus after the run, so editing goes on.
- Plain Enter, and Enter or Tab while the suggestion list is open, keep their current meaning.
- The Run button and the shortcut share one code path, so they can't drift apart.
- The Run button's tooltip names the shortcut, and the Script paragraph of
docs/user-guide/analysis.mdmentions it. - A test checks that Ctrl+Enter with the focus in the box runs the script and leaves the text unchanged.
Estimated scope
Small: a minor tweak, small UI change, or text adjustment
Alternatives considered
- Shift+Enter, as in notebooks. Ctrl+Enter matches the SQL pane, and one shortcut for "run what I typed" across panes is easier to learn.
- A global keymap entry (
keymap.rs). The shortcut only makes sense while the Script box has the focus, so it belongs with the box, as in the SQL pane.
- Dominant language
- Rust
- Stars
- 1.9k
- Forks
- 268
- Avg merge
- 3h 1m
- Merged PRs (30d)
- 556
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from opengeos/RustGIS
-
enhancement enhancement (small)
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 1 day
-
enhancement (small) good first issue
Difficulty 4/5 3-5 days Newbie friendliness 55/100
Maintainers usually reply within 1 day
-
enhancement wishlist
Difficulty 5/5 Over a week Newbie friendliness 25/100
Maintainers usually reply within 1 day
-
[Bug]: Unique Values symbology fails with "field is required" for layers without numeric fieldsPossibly taken @deniial00 claimed this today. Openunconfirmed possible issue
opengeos/RustGIS#768 · 1 assignee ·
Maintainers usually reply within 1 day
-
[Feature]: Read every feature of an ArcGIS FeatureServer or MapServer layer, past its record limitPossibly taken @Mavrick-F claimed this today. Openenhancement
opengeos/RustGIS#764 · 1 assignee ·
Maintainers usually reply within 1 day
All issues in opengeos/RustGIS
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day
-
feature
Difficulty 1/5 Under an hour Newbie friendliness 92/100
dsx-ai-factory/infra-controller#8145 ·
Maintainers usually reply within 1 day
-
bug: workspace access hint suggests a --subject command that fails for hyphen-leading OIDC subjectsOpen
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 1 day
-
type: bug
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 1 day
-
A-FFI C-Testing D-Straightforward S-Ready
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
Maintainers usually reply within 1 day