dart-pad icon indicating copy to clipboard operation
dart-pad copied to clipboard

support multiple keybindings for some actions

Open devoncarew opened this issue 4 months ago • 0 comments

  • support multiple keybindings for some actions
  • run is now cmd-r (as well as the older dartpad binding, cmd-enter); fix https://github.com/dart-lang/dart-pad/issues/2818
  • format is now cmd-s (as well as the vs code binding, shift-option-f); fix https://github.com/dart-lang/dart-pad/issues/2848
  • added support for the older dartpad binding for quick fixes - option-enter; address part of https://github.com/dart-lang/dart-pad/issues/2868

Note that the behavior of cmd-s changes with this PR; before, it runs the snippet, after, it formats it. cmd-r is probably better as a run keybinding (and will probably prevent people from unintentionally reloading the dartpad page), and cmd-s should be familiar to vs code users as many have format-on-save active.

Screenshot 2024-02-29 at 8 06 40 AM
  • [x] I’ve reviewed the contributor guide and applied the relevant portions to this PR.
Contribution guidelines:

Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

devoncarew avatar Feb 29 '24 16:02 devoncarew