code icon indicating copy to clipboard operation
code copied to clipboard

Move Strip Trailing Space from the Extensions tab to Behavior or Interface

Open mks-h opened this issue 3 years ago • 6 comments

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.

mks-h avatar Sep 06 '21 16:09 mks-h

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.

jeremypw avatar Sep 06 '21 16:09 jeremypw

I agree that some extension functionality like "Preserve Indent" could be absorbed into the main code base as settings.

jeremypw avatar Sep 06 '21 16:09 jeremypw

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 avatar Sep 17 '21 01:09 igordsm

@igordsm Its :+1: 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.

jeremypw avatar Sep 19 '21 09:09 jeremypw

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.

Marukesu avatar Sep 30 '21 20:09 Marukesu

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.

jeremypw avatar Dec 08 '21 18:12 jeremypw