-
Notifications
You must be signed in to change notification settings - Fork 676
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
Add option for formatting code block open braces #6939
Add option for formatting code block open braces #6939
Conversation
"type": "boolean", | ||
"scope": "window", | ||
"default": false, | ||
"description": "Forces the open brace after an @code or @functions directive to be on the following line." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you expect this to be localized?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fine with the change to be Eng only right now. Seems other options are also not localized. So it might be good to do that in another PR.
Thanks for the heads up, I just copied the existing pattern and didn't really think about it. Logged dotnet/razor#10025 to fix this in another PR. |
…`@code` or `@functions` block (#10018) Fixes: Being nerdsniped by `u/featheredsnake` on Reddit: https://www.reddit.com/r/Blazor/comments/1b47odh/am_i_the_only_one_bothered_that_code_brackets/ ![27c833c0-aa9c-4be6-9519-3b3ecb9f3f54](https://github.com/dotnet/razor/assets/754264/d3c16ea4-9b75-4972-a4bd-ad4bf986b54d) VS Code option: dotnet/vscode-csharp#6939
bcd171e
to
0fe06a4
Compare
VS Code option for dotnet/razor#10018