Copy As Html doesn't copy C# or F# colour about vs-ppt

[olegtk]

I have a fix for CopyAsHTML, will push an update soon.
The root cause turned to be this change in Roslyn behavior: https://developercommunity.visualstudio.com/t/ITagAggregatorIClassificationTag-no-lo/10595181

[ploeh]

ping

[ploeh]

Here's another observation: The setting UnindentToRemoveExtraLeadingWhitespace is set to True, which I believe is the default. This setting, however, is not consistently applied.

Here's what I highlight:

Here's what I get:

        double value1 = 100.00D;
double value2 = 15.99D;
double result = client.Add(value1, value2);

[ploeh]

I've managed to reset the extension to use default values (i.e. defaults for BeforeCodeSnippet and AfterCodeSnippet), but that doesn't help.

[BertanAygun]

Hi @ploeh, can you confirm if you have this option enabled:

We added support for accurate classification for more accurate colors based on Tools -> Options -> Text Editor -> Advanced -> "Use accurate classifier" setting. Please note that copy operations can be slower when this setting is enabled.

Thanks,
Bertan

[ploeh]

It's getting late here in Copenhagen, so it's possible that I'm just too tired, but where do I find that setting?

[BertanAygun]

I also just realized that setting no longer exists in product so I will have to investigate and reply back here.

Thanks,
Bertan

[olegtk]

"Use accurate classifier" setting was really bad perf-wise in general case and was removed in 17.9. I suggest either just make CopyAsHTML always call accurate classifier or introduce it's own CopyAsHTML-specific option to control whether to use accurate classification, and make it default.

[ploeh]

I have a fix for CopyAsHTML, will push an update soon.

Looking forward to it!

The root cause turned to be this change in Roslyn behavior: https://developercommunity.visualstudio.com/t/ITagAggregatorIClassificationTag-no-lo/10595181

Ooh, a breaking change in a Microsoft product!

[ploeh]

Any news on when we can expect the update?

[iccfish]

Seems this extension out of maintenance?

[heiligerdankgesang]

Thanks for following up, folks. While we do have a fix, there’s some additional work that needs to be completed before we can update the extension with the fix and make it available for use. We don’t yet have a timeline for release, but please keep an eye on this issue for further updates.