-
-
Notifications
You must be signed in to change notification settings - Fork 116
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
Move Strip Trailing Space from the Extensions tab to Behavior or Interface #1098
Comments
They are extensions inasmuchas they are provided by separate modules through a plugin architecture, not by the main codebase. So, in theory at least, there could be third-party plugins in the list. |
I agree that some extension functionality like "Preserve Indent" could be absorbed into the main code base as settings. |
I'm working on a Snippet functionality and having "Preserve Indent" (and its functions) in the main code base would be very convenient. Some of the code would be useful when inserting snippets with multiple lines. @jeremypw I could work on this refactoring, is it desirable? |
@igordsm Its 👍 from me. Maybe best to raise a specific issue first to get wider feedback in case there is a drawback I am not aware of. |
I'm working in add support for the rest of the editorconfig spec, and it would be good to have "Strip trailing whitespace" as part of code itself. |
I am going to focus this issue on "Strip Trailing Spaces". Please open new issues for each extension to be moved into the main code so they can be addressed one at a time. |
Problem
The so-called extensions are not really extensions but just checkboxes for a particular feature.
Proposal
They better be proper switches in Behavior or Interface tabs.
The text was updated successfully, but these errors were encountered: