Skip to content

Adding a Ctrl+I hotkey to reindent all lines (following code from https://groups.google.com/d/msg/codemirror/oGeWPzZynxo/cJoQMzkj0QoJ) #50

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 1, 2016

Conversation

blerner
Copy link
Member

@blerner blerner commented Feb 20, 2016

This should provide an equivalent experience to WeScheme's reindent-all. It's bound to Ctrl+I, which is already a hotkey in (at least) Firefox, but as long as the editor has the keyboard focus, it works fine. Tested on CPO/Pyret-master, and it works for me.

@blerner blerner merged commit dd93baf into master Mar 1, 2016
@shriram
Copy link
Member

shriram commented Mar 1, 2016 via email

@blerner
Copy link
Member Author

blerner commented Mar 1, 2016

That's a general question -- all of our keyboard UI is rather undiscoverable. This particular hotkey is identical to WeScheme, so it should make @schanzer happy that it's now merged, but yes, we certainly need to improve our hotkey help story.

@shriram
Copy link
Member

shriram commented Mar 1, 2016

Could the More menu have a Hotkeys entry that pops up a modal "window"
showing the current hotkeys?

On Tue, Mar 1, 2016 at 11:39 AM, Ben Lerner notifications@github.com
wrote:

That's a general question -- all of our keyboard UI is rather
undiscoverable. This particular hotkey is identical to WeScheme, so it
should make @schanzer https://github.com/schanzer happy that it's now
merged, but yes, we certainly need to improve our hotkey help story.


Reply to this email directly or view it on GitHub
#50 (comment)
.

@blerner blerner deleted the reindent-all branch June 26, 2016 12:27
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.

2 participants