-
Notifications
You must be signed in to change notification settings - Fork 11
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
Indentation Rules #306
Indentation Rules #306
Conversation
Let's merge this PR and open a couple issues: One for the fact that it doesn't work going up, and another to simplify the logic to use the same "change indentation" logic across the board. With this PR we'll have duplicated logic across the formatter and this code, won't we? I'd still like to merge this PR as it's an improvement over the current behavior, right? |
Agree.
Not agree. Formatter can't share this code at all. It's internal VS Code implementation. Theoretically we can implement in AHK++ On big scripts (>1k lines) there are noticible time execution of regular formatter. It can't do it in realtime.
Yes, it's very fast realtime formatting by VS Code implementation, why not use it :) |
Created issue #319. |
Sounds good, I'm still not 100% convinced we can't find some way to share this logic, but that's not a big deal. Thanks again for all your work on this project :) |
Changes proposed in this pull request:
indentationRules
toahk.configuration.json
VS Code should adjust the indentation of current line or next line when you type, paste, and move lines.
Example:
Raw text:
RegEx capture correctly, but VS Code won't work correctly on moving line upward. Any suggestions?
Indent +
Indent -
Can't make regex case insensitive:
(?i:)
not workIndentation Rules
Notifying @mark-wiemer