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

[BUG] When using C# Dev Kit, code inserted by quick fixes doesn't respect tab size. #1651

Open
Nixill opened this issue Dec 2, 2024 · 4 comments
Assignees
Labels
area-roslyn bug Something isn't working
Milestone

Comments

@Nixill
Copy link

Nixill commented Dec 2, 2024

Describe the Issue

I code with a tab size of two spaces. However, any code that a C# Dev Kit Quick Fix modifies becomes four-spaced.

Steps To Reproduce

  1. Write some code
  2. Make sure tab size is "Spaces: 2"
  3. Extract a part of that code to a method
  4. The code extracted to a method, as well as the code near where the new method was placed, becomes indented with four-space tabs.

Expected Behavior

The extracted code should be indented with two-space tabs.

Environment Information

Version: 1.96.0-insider (user setup)
Commit: 275faf6f08b7aa50843f3c18406b4d5969784e52
Date: 2024-11-28T18:04:55.375Z
Electron: 32.2.6
ElectronBuildId: 10629634
Chromium: 128.0.6613.186
Node.js: 20.18.1
V8: 12.8.374.38-electron.0
OS: Windows_NT x64 10.0.19045

C# Dev Kit 1.14.12
@Nixill Nixill added the bug Something isn't working label Dec 2, 2024
@dibarbet
Copy link
Member

dibarbet commented Dec 4, 2024

@Nixill if you turn off the vscode setting editor.DetectIndentation, does it respect your vscode spaces setting (editor.tabSize)?

@dibarbet dibarbet added the needs-more-info Needing more information, waiting on OP label Dec 4, 2024
@Nixill
Copy link
Author

Nixill commented Dec 8, 2024

It does. I'm assuming that's an already-filed bug somewhere else?

@github-actions github-actions bot removed the needs-more-info Needing more information, waiting on OP label Dec 8, 2024
Copy link

github-actions bot commented Dec 8, 2024

@dibarbet, the 'needs-more-info' label has been removed upon receiving further response from the original bug filer.

@dibarbet
Copy link
Member

dibarbet commented Jan 4, 2025

We will use this one to track that issue - thanks for the confirmation.

@dibarbet dibarbet self-assigned this Jan 4, 2025
@dibarbet dibarbet added this to the Jan2025 milestone Jan 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-roslyn bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants