search

bsivanov / ClearTrackedChanges

The source code of Clear Tracked Changes extension for Visual Studio.
https://marketplace.visualstudio.com/items?itemName=bsivanov.ClearTrackedChanges
GNU General Public License v3.0
7 stars 0 forks source link

Visual Studio 2017 doesn't clear aspx #1

Open goodeye opened 5 years ago

goodeye commented 5 years ago

Hi, I've used your extension for a long time with success. I just upgraded to VS 2017, and it works on most page types. It is not working for .aspx. It is working for all others, e.g., .cs, aspx.cs, and even ascx.

I see it might be a visual studio issue: if you manually uncheck the option (Tools -> Options -> Text Editor -> Track Changes), the bars go away, but when checked again, the bars are restored. I can't find a bug report on Visual Studio for this (but it's hard to search this issue).

This is Visual Studio 2017 version 15.9.4 on Windows 7 and Windows 10, with .NET 4.7.03056

Thanks, Bob

bsivanov commented 5 years ago

I can confirm this. The extension works more as a cheat - behind the scenes, it just toggles the option you mentioned. It seems that the .aspx editor though is smarter and preserves the tracked changes when the "Track Changes" option is toggled.

I guess there is more intelligent way to implement this extension - by clearing the tags responsible for preserving the tracked changes directly in the current editor, but I am not sure how. More info on editor extensibility could be found here: Inside the Editor -> Tags and classifiers.