starred/terminal
1
0
Fork 0
mirror of https://github.com/microsoft/terminal.git synced 2026-02-08 13:49:31 +00:00
Code Issues 19.0k Packages Projects Releases 20 Wiki Activity

this certainly makes it a bit better

Browse Source
This commit is contained in:
Mike Griese
2022-03-16 13:10:37 -05:00
parent ee83081b64
commit 1005e0dfc5
1 changed files with 16 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
src/cascadia/TerminalSettingsEditor/Profiles_Appearance.cpp
View File

@@ -37,24 +37,34 @@ namespace winrt::Microsoft::Terminal::Settings::Editor::implementation
ProfileViewModel::UpdateFontList();
}
auto weakThis{ get_weak() };
// Subscribe to some changes in the view model
// These changes should force us to update our own set of "Current<Setting>" members,
// and propagate those changes to the UI
_ViewModelChangedRevoker = _Profile.PropertyChanged(winrt::auto_revoke, [=](auto&&, const PropertyChangedEventArgs& /*args*/) {
_previewControl.UpdateControlSettings(_Profile.TermSettings(), _Profile.TermSettings());
_ViewModelChangedRevoker = _Profile.PropertyChanged(winrt::auto_revoke, [weakThis](auto&&, const PropertyChangedEventArgs& /*args*/) {
if (auto page{ weakThis.get() })
{
page->_previewControl.UpdateControlSettings(page->_Profile.TermSettings(), page->_Profile.TermSettings());
}
});
// The Appearances object handles updating the values in the settings UI, but
// we still need to listen to the changes here just to update the preview control
_AppearanceViewModelChangedRevoker = _Profile.DefaultAppearance().PropertyChanged(winrt::auto_revoke, [=](auto&&, const PropertyChangedEventArgs& /*args*/) {
_previewControl.UpdateControlSettings(_Profile.TermSettings(), _Profile.TermSettings());
_AppearanceViewModelChangedRevoker = _Profile.DefaultAppearance().PropertyChanged(winrt::auto_revoke, [weakThis](auto&&, const PropertyChangedEventArgs& /*args*/) {
if (auto page{ weakThis.get() })
{
page->_previewControl.UpdateControlSettings(page->_Profile.TermSettings(), page->_Profile.TermSettings());
}
});
// There is a possibility that the control has not fully initialized yet,
// so wait for it to initialize before updating the settings (so we know
// that the renderer is set up)
_previewControl.Initialized([&](auto&& /*s*/, auto&& /*e*/) {
_previewControl.UpdateControlSettings(_Profile.TermSettings(), _Profile.TermSettings());
_previewControl.Initialized([weakThis](auto&& /*s*/, auto&& /*e*/) {
if (auto page{ weakThis.get() })
{
page->_previewControl.UpdateControlSettings(page->_Profile.TermSettings(), page->_Profile.TermSettings());
}
});
}