Skip to content
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

[TextEditorOptions] support end of line #3817

Closed
SamVerschueren opened this issue Mar 8, 2016 · 10 comments
Closed

[TextEditorOptions] support end of line #3817

SamVerschueren opened this issue Mar 8, 2016 · 10 comments
Assignees
Labels
feature-request Request for new features or functionality verified Verification succeeded
Milestone

Comments

@SamVerschueren
Copy link
Contributor

In the EditorConfig plugin, we want to implement end_of_line as well. In the latest VS Code release, I noticed that we have a files.eol setting now. Would it be possible to expose this to the TextEditorOptions interface to set this on the fly?

// @alexandrudima

@alexdima
Copy link
Member

alexdima commented Mar 9, 2016

@SamVerschueren I've just added a way to do that this morning via textEditor.edit().

We thought to put it on the editBuilder since it changes the document content.

fyi @jrieken

@alexdima alexdima added this to the March 2016 milestone Mar 9, 2016
@alexdima alexdima added the feature-request Request for new features or functionality label Mar 9, 2016
@alexdima
Copy link
Member

alexdima commented Mar 9, 2016

The relevant commit is 45906d9

@alexdima alexdima closed this as completed Mar 9, 2016
@SamVerschueren
Copy link
Contributor Author

Awesome, thanks! The only thing that's missing for an entire EditorConfig integration is charset. There is already an open issue for this #824

@alexdima
Copy link
Member

fyi @bpasero editor config can contain charset configurations.

Also missing from the API: trim_trailing_whitespace and insert_final_newline, but I think there are workarounds for these last two (submitting edits to the text editor).

@bpasero
Copy link
Member

bpasero commented Mar 10, 2016

@alexandrudima would it make sense to have charset on the model and me picking it up from there?

@alexdima
Copy link
Member

@bpasero I don't think that would work, because the charset should be used when creating the model, it would be too late to have it as a setting on a model.

@alexdima alexdima assigned jrieken and unassigned alexdima Mar 21, 2016
@SamVerschueren
Copy link
Contributor Author

Maybe we should track that in a different issue?

@jrieken jrieken added the verified Verification succeeded label Mar 23, 2016
@SamVerschueren
Copy link
Contributor Author

Is an API for changing charset somewhere on the roadmap? It's the last missing piece for EditorConfig.

@jrieken
Copy link
Member

jrieken commented Apr 15, 2016

No plans yet, will also be a little extra work as for us. I have created #5318 for taht

@SamVerschueren
Copy link
Contributor Author

Thanks @jrieken

@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

4 participants