-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Make InteractiveWindow use HACK_ThemeColorFixer #5435
Conversation
Fix issue 4930 dotnet#4930
@@ -52,6 +52,8 @@ protected override void Initialize() | |||
|
|||
_componentModel = (IComponentModel)GetService(typeof(SComponentModel)); | |||
_interactiveWindowProvider = _componentModel.DefaultExportProvider.GetExportedValue<TVsInteractiveWindowProvider>(); | |||
KnownUIContexts.ShellInitializedContext.WhenActivated(() => | |||
_componentModel.GetService<HACK_ThemeColorFixer>()); |
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.
@genlu does this change only affect the pop ups?
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.
@ManishJayaswal based on the code of HACK_ThemeColorFixer
, it affects both tooltip and immediate , although I don't know what an immediate is...
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.
The "immediate" IClassificationFormatMap
that we're updating is for the Immediate Window (eventually mapping down to DefGuidList.guidImmediateWindowFontCategory
.
@dotnet-bot retest this, please |
👍, assuming you manually verified that it works. |
👍 |
Make InteractiveWindow use HACK_ThemeColorFixer
Fix issue #4930
@amcasey @dpoeschl @tmat @cston @KevinH-MS @ManishJayaswal