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

New .editorconfig designer does not save changes #54556

Open
vsfeedback opened this issue Jul 1, 2021 · 35 comments
Open

New .editorconfig designer does not save changes #54556

vsfeedback opened this issue Jul 1, 2021 · 35 comments
Assignees
Labels
Area-IDE Bug Developer Community The issue was originally reported on https://developercommunity.visualstudio.com Feature - Editor Config UI Urgency-Soon
Milestone

Comments

@vsfeedback
Copy link

This issue has been moved from a ticket on Developer Community.


git clone https://[email protected]/devdiv/DevDiv/_git/VS.CloudCache
cd VS.CloudCache
git checkout 2c9b11133a2fc8acfb9c73486d1fa5cca0703821
.\Microsoft.VisualStudio.Cache.sln

In Solution Explorer, expand the test folder and open .editorconfig.
In the designer, make a change (check a box, or in my case set SA1611 to Disabled
Save the document and close.

Expected

Git Changes shows the file has been changed on disk.

Actual

No change is applied to the file.

And BTW, since this designer now preempts the text editor opening the document, this is a regression in my ability to maintain these files from within VS.

Workaround

Use Open With... to force the text editor to open the document.


Original Comments

Feedback Bot on 5/12/2021, 07:25 PM:

We have directed your feedback to the appropriate engineering team for further evaluation. The team will review the feedback and notify you about the next steps.

Aimee Cao [MSFT] on 5/24/2021, 02:58 AM:

Thank you for your feedback!
We were unable to open the .editorconfig file in your repo and it showed up blank. we verified the .editorconfig file in the other repo and after modifying it, it showed up in the git changes window. Could you give us a simple repo that reproduces this issue?
Do other types of files show up in the git changes window after they’ve been modified?

We are looking forward to hearing from you soon.
Thanks
Why do we ask for more info?
We try to reproduce all issues reported with the information provided in the description and comments. When we can’t reproduce the issue, we ask you for more information so we can resolve the issue as quickly and efficiently as possible.
In our guidelines, you can get tips on how to provide clear and simple reproducible steps.

Andrew Arnott [MSFT] on 5/24/2021, 07:28 AM:

This no longer repros as of 31324.4.main.

Feedback Bot on 6/2/2021, 07:21 AM:

We have directed your feedback to the appropriate engineering team for further evaluation. The team will review the feedback and notify you about the next steps.

Feedback Bot on 6/2/2021, 09:41 AM:

Thank you for sharing your feedback! Our teams prioritize action on product issues with broad customer impact. See details at: https://docs.microsoft.com/en-us/visualstudio/ide/report-a-problem?view=vs-2019#faq. In case you need answers to common questions or need assisted support, be sure to use https://visualstudio.microsoft.com/vs/support/. We’ll keep you posted on any updates to this feedback.

Rasmus Wätjen on 6/7/2021, 00:51 AM:

I have this issue also. For me it is only when changing severity from Warning or Error to Suggestion.
In that case the file does not change on disk, and the file does not show as changed in the IDE.
When changing from Suggestion to Warning or Error it works as expected.

Also: If I change the severity FROM Suggestion to e.g. Warning, and change it back - without saving in the mean time, then the when I save the file, the severity is stored as Warning.


Original Solutions

Aimee Cao [MSFT] solved on 5/24/2021, 06:48 PM, 0 votes:

Thank you for your feedback! I’m going to close this as not a bug, if you encounter the issue again, please re-active it or summit a new one. Thanks for contributing to Developer Community.

Andrew Arnott [MSFT] on 6/2/2021, 07:19 AM:

Actually, this is still repro’ing. I’m on 31401.7.main now and the designer never saves changes I make to analyzer severity settings.

Jonathan Griffiths solved on 6/29/2021, 02:29 AM, 0 votes:

[Workaround]
This is a pain. The easiest workaround I’ve found is to expand the Solution Explorer > Dependencies > Analyzers and then to right-click on the rules in turn and set the severity. This seems to update the .editorConfig file.

@mavasani mavasani transferred this issue from dotnet/roslyn-analyzers Jul 2, 2021
@dotnet-issue-labeler dotnet-issue-labeler bot added Area-IDE untriaged Issues and PRs which have not yet been triaged by a lead labels Jul 2, 2021
@mavasani
Copy link
Contributor

mavasani commented Jul 2, 2021

@jmarolf

@Youssef1313
Copy link
Member

This is part of #53171 (the 7th point)

@light-traveller
Copy link

light-traveller commented Apr 19, 2022

Any ETA about this bug?
It is badly affecting the work.

@light-traveller
Copy link

light-traveller commented Apr 19, 2022

I am also getting the following error when adding a .editorconfig file to a blank solution.
image

This is the VS version info:
image

@jinujoseph jinujoseph modified the milestones: 17.2, 17.3 May 5, 2022
@Clonkex
Copy link

Clonkex commented May 6, 2022

There's at least three issues open for this problem (this one, #56676 and #60232) and somehow it's still an issue after nearly a year. I don't get how major software like this can ship with totally non-functional UI (over multiple versions!). It's not like it's a minor usability issue, it's literally just completely broken and useless. I also don't see how this is a Roslyn issue rather than a Visual Studio issue (but I have no idea how things are implemented internally so maybe it makes sense).

FWIW I'm seeing this issue while trying to set "Use Tabs" to true, but nothing changes in the file.

@light-traveller
Copy link

light-traveller commented May 7, 2022

There's at least three issues open for this problem (this one, #56676 and #60232) and somehow it's still an issue after nearly a year. I don't get how major software like this can ship with totally non-functional UI (over multiple versions!). It's not like it's a minor usability issue, it's literally just completely broken and useless. I also don't see how this is a Roslyn issue rather than a Visual Studio issue (but I have no idea how things are implemented internally so maybe it makes sense).

FWIW I'm seeing this issue while trying to set "Use Tabs" to true, but nothing changes in the file.

As far as I have seen throughout the years, Microsoft almost always acts like low-budget start-ups in releasing software-even for commercial products like VS 2022! It first releases a problematic software, then waits for the community to find bugs and report issues! This makes no sense whatsoever.

How come such a wealthy huge corporation does not posses enough resources to do a clean release for God's sake? Seriously how?

@CyrusNajmabadi
Copy link
Member

@light-traveller we are a team with a given amount of resources, just like any project. Currently there are roughly 5k+ issues to get through, so that means at any time there are likely many thousands of things that don't make the cut to get fixed given the manpower and time we have.

If these issues are very important to you though, we are an open source project and would gladly accept help and contributions here like we do for many other areas. That can allow you to get things prioritized to your need level as opposed to the need of the entire community.

Thanks!

@drizztdourden08
Copy link

I've seen bugs reported for this for OVER 2 years... Seriously. How the hell is a simple "Your shit doesn't work and save the file" not fixed at this point. There is not a single reason in this world that will justify this.

NONE.

Microsoft has enough resources for this to happen. No matter what the current team workload is, it's your job to report that you don't have enough resources and ask for more so us, the clients, can have the basics working.

@Clonkex
Copy link

Clonkex commented Mar 9, 2023

It'd be great if they at least investigated and let us know what the problem is. Either the problem is easy to fix, in which case they can fix it, or it's hard to fix, in which case it'd be nice to know exactly why it's hard to fix. Anything to justify it being broken for so long.

@drizztdourden08
Copy link

Seriously, what could be hard about this?

1- Populate the file from something (Actually don't know what that is)
2- Read the config from the file and bind the UI.
2- Let the user choose whatever.
3- Write the said lines to the file.

Done.

This is an easy job for even the newest of programmers.

I just solved the issues on my end by opening the file using "Open with", erasing literally everything. There was a class {} in it somehow. Afterward, everything worked just fine.

I'm pretty sure this is just a matter of checking if the source file is actually compatible and valid. Cause after that, it wrote everything as expected. How is that not a thing that's checked literally when the file is opened is beyond me.

@CyrusNajmabadi
Copy link
Member

@drizztdourden08 could you provide a repro that demonstrates the issue?

@drizztdourden08
Copy link

@drizztdourden08 could you provide a repro that demonstrates the issue?

In a new project, create a file at the root using right click -> Add -> name it .editorconfig -> I didn’t choose anything thinking it’d be blank but c# class was selected so the template for this was in the file.

Open .editorconfig with double click, the ui show and populate normally but no matter what changes, there is no * beside the file name to signify you can save.

I opened it with “Open with”, then saw the tenplate for a c# class file. Deleted everything. Solved.

@CyrusNajmabadi
Copy link
Member

@drizztdourden08 I tried the above and had no issues saving changes. Something must be different about your setup. Any chance you could put your project on GitHub and give us a link?

@drizztdourden08
Copy link

drizztdourden08 commented Mar 9, 2023 via email

@CyrusNajmabadi
Copy link
Member

Hi @drizztdourden08 , that's very helpful. We'll definitely try out on a Unity project to see if it reproes and then indicates what the problem is. Thanks for sharing that information!

@kirsan31
Copy link

@CyrusNajmabadi
Point 2 from my post is 100% reproducible, and may be global problem is related to it...

I agree with @drizztdourden08 that this UI editor is not usable from beginning :( For me it's mainly due to this (#54556) and #59325 bugs...

@makimusan
Copy link

Hello @vsfeedback ,
I've faced with a similar issue. When I was adding edc file via solution explorer then changes hasn't been saved for the edc file. It can be resolved used follow steps:

  • go to Tool -> Options
  • go to Text Editor -> C# -> Code Style
  • press the Generate .editorconfig file from settings button
  • choose a folder and save the file
    It was weird case but standard methods didn't work only for this one project for others everything was fine. I hope you find my experience useful.

@vchtelmakh
Copy link

I have the same issue. Basic .NET 6 API project. .editorconfig does not save. Visual Studio is entirely up to date as of 03/25/2024.

@bcheung4589
Copy link

bcheung4589 commented May 6, 2024

Shit is STILL not solved. But I guess who cares about the editor config of an premium product where the UI is made by an hobbist.. Who earns money on this project? And has the fkn balls to ask the community to deliver free work, even worse, almost expected.. I wonder how long that kind of community abuse will work..

"Crashes, hangs, data loss, etc. would all have higher priority."
~ 3 years.. lets see what excuses are made now.. because at my job, we use planning, we can handle crashes and breaking UI in the same year. Sometimes they are correlated, so we dont make excuses, we make solutions (also no 50% solutions).

@CyrusNajmabadi
Copy link
Member

@bcheung4589 we'd welcome contributions here. Thanks!

@bcheung4589
Copy link

@bcheung4589 we'd welcome contributions here. Thanks!

Thats earned. People dont just go contribute their time and effort on projects. If I open our project up to public, the company I work for would welcome your contributions to the project also.

@CyrusNajmabadi
Copy link
Member

People dont just go contribute their time and effort on projects.

Lots of people do. Dotnet and Roslyn both have lots of external contributors. That's the great thing about open source :)

If you're interested, let us know and we'll happily work with you to help you get ramped up here. If not, we understand. Have a good day! :)

@ShadyHarbinger
Copy link

Same issue here VS 2022 .NET 8 SDK projects. Cannot modify .editorconfig files. Always resets to default values.

@JanKotschenreuther
Copy link

JanKotschenreuther commented Jul 26, 2024

Saving changes fails, if the .editorconfig does NOT live in a project directory.
Saving changes succeeds, if the .editorconfig lives in a project directory.

For most repositories we place the .editorconfig at the root of the repository, to enforce the same rules for all projects/sub-directories.
For most of our repositories, the root is also where the solution file lives.
Every project lives in a sub directory.

Using VS Enterprise 2022 (64-bit) v17.10.4

ryzngard added a commit that referenced this issue Sep 4, 2024
@arunchndr arunchndr modified the milestones: 17.6, Backlog Oct 28, 2024
@arunchndr arunchndr assigned sharwell and unassigned jmarolf Oct 29, 2024
@arunchndr arunchndr modified the milestones: Backlog, 17.13 P2 Oct 29, 2024
@LeDahu22
Copy link

This is still happening in VS 2022 17.12.1...

@bcheung4589

This comment has been minimized.

@LeDahu22
Copy link

I think I managed to find the issue on my side, there were duplicate rules in my editorconfig (duplicate were created by another member of the team, which edited the file manually).

When there were duplicates :

  • First occurence was updated and the second wasn't.
  • But when the file was being read, the last one was taken into account.

So, I deleted my editorconfig file, and started from scratch with a fresh one generated by VS. Now everything's working again (I had to reconfigure few rules, but it's not a big deal).

@bcheung4589
Copy link

Great find! Afraid it wasnt my problem, didnt find any duplicates :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-IDE Bug Developer Community The issue was originally reported on https://developercommunity.visualstudio.com Feature - Editor Config UI Urgency-Soon
Projects
None yet
Development

No branches or pull requests