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

editor.formatOnType does not respect editor.insertSpaces setting #834

Closed
aeschli opened this issue Oct 21, 2016 · 5 comments · Fixed by #1055
Closed

editor.formatOnType does not respect editor.insertSpaces setting #834

aeschli opened this issue Oct 21, 2016 · 5 comments · Fixed by #1055
Assignees
Milestone

Comments

@aeschli
Copy link
Contributor

aeschli commented Oct 21, 2016

From @j-h-a on October 20, 2016 15:58

  • VSCode Version:
    1.6.1
  • OS Version:
    OSX (10.11.6)

Searchable description

VS Code keeps changing some lines to have spaces instead of tabs in the leading whitespace.

Steps to Reproduce:

Use these settings:
"editor.tabSize": 4,
"editor.insertSpaces": false,
"editor.detectIndentation": false,
"editor.formatOnType": true,

Type some code that triggers formatOnType to activate a format change.

Expected behaviour:

Line is formatted using tabs for leading whitespace, because of the insertSpaces setting.

Actual behaviour:

Line is formatted with spaces instead of tabs for the leading whitespace.

Copied from original issue: microsoft/vscode#14108

@aeschli
Copy link
Contributor Author

aeschli commented Oct 21, 2016

From @roblourens on October 20, 2016 18:25

Is this in an html file? Could be #13563

@aeschli
Copy link
Contributor Author

aeschli commented Oct 21, 2016

From @j-h-a on October 20, 2016 18:28

No, this is while editing C#

@aeschli
Copy link
Contributor Author

aeschli commented Oct 21, 2016

From @roblourens on October 20, 2016 18:35

Related @aeschli?

@shawnwildermuth
Copy link

Any word on status of this issue? I still cant' seem to find a solution.

@filipw
Copy link
Contributor

filipw commented Dec 18, 2016

@shawnwildermuth here is a PR #1055

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants