-
Notifications
You must be signed in to change notification settings - Fork 5
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
Quickscript #34
Comments
I don't think it should call the Racket indenter, so the current state is exactly what I want. Thanks again! |
Note though that DrRacket has rules in |
I think the better fix is to make all What I don't want to happen is DrRacket undoes correct indentation by |
How about |
The problem is that DrRacket rules provide a lot less information. Take
The indentation rule in DrRacket only says that whatever comes after the first line of Both:
and
are admissible formats consistent with the indentation rule in DrRacket. But |
Well, that's because it's an indenter, not a formatter as you well know. That doesn't mean that DrRacket's indentation rules should be entirely disregarded. At least, is it possible to add the default indentation rules of DrRacket, even if we don't build a bridge between the two? |
Any traction on this? I use
and am just now discovering that
|
I added a basic support for |
Here's a slightly improved quickscript:
https://gist.github.com/Metaxal/f45ea1a893a1bdedaadbeafd11144e3d
It formats either the selection, or the whole program if the selection is empty.
It should probably call DrRacket's indenter afterwards, but I couldn't find the API for it, which means I would need to dig the keybinding function string name, get the keymap, etc.
The text was updated successfully, but these errors were encountered: