Skip to content

Commit

Permalink
raise the event relative to the sender
Browse files Browse the repository at this point in the history
  • Loading branch information
zadjii-msft committed Aug 8, 2023
1 parent 0cf6475 commit 902ee8e
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 8 deletions.
14 changes: 8 additions & 6 deletions src/cascadia/TerminalApp/TerminalPage.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -4691,7 +4691,7 @@ namespace winrt::TerminalApp::implementation
_updateThemeColors();
}

winrt::fire_and_forget TerminalPage::_ControlCompletionsChangedHandler(const IInspectable /*sender*/,
winrt::fire_and_forget TerminalPage::_ControlCompletionsChangedHandler(const IInspectable sender,
const CompletionsChangedEventArgs args)
{
// This will come in on a background (not-UI, not output) thread.
Expand All @@ -4713,20 +4713,22 @@ namespace winrt::TerminalApp::implementation
args.ReplacementLength());

auto weakThis{ get_weak() };
Dispatcher().RunAsync(CoreDispatcherPriority::Normal, [weakThis, commandsCollection]() {
Dispatcher().RunAsync(CoreDispatcherPriority::Normal, [weakThis, commandsCollection, sender]() {
// On the UI thread...
if (const auto& page{ weakThis.get() })
{
// Open the Suggestions UI with the commands from the control
page->_OpenSuggestions(commandsCollection, SuggestionsMode::Menu);
page->_OpenSuggestions(sender.try_as<TermControl>(), commandsCollection, SuggestionsMode::Menu);
}
});
}
CATCH_LOG();
}

void TerminalPage::_OpenSuggestions(IVector<Command> commandsCollection,
winrt::TerminalApp::SuggestionsMode mode)
void TerminalPage::_OpenSuggestions(
const TermControl& sender,
IVector<Command> commandsCollection,
winrt::TerminalApp::SuggestionsMode mode)
{
// ON THE UI THREAD
assert(Dispatcher().HasThreadAccess());
Expand All @@ -4744,7 +4746,7 @@ namespace winrt::TerminalApp::implementation
return;
}

const auto& control{ _GetActiveControl() };
const auto& control{ sender ? sender : _GetActiveControl() };
if (!control)
{
return;
Expand Down
2 changes: 1 addition & 1 deletion src/cascadia/TerminalApp/TerminalPage.h
Original file line number Diff line number Diff line change
Expand Up @@ -519,7 +519,7 @@ namespace winrt::TerminalApp::implementation
void _updatePaneResources(const winrt::Windows::UI::Xaml::ElementTheme& requestedTheme);

winrt::fire_and_forget _ControlCompletionsChangedHandler(const winrt::Windows::Foundation::IInspectable sender, const winrt::Microsoft::Terminal::Control::CompletionsChangedEventArgs args);
void _OpenSuggestions(Windows::Foundation::Collections::IVector<winrt::Microsoft::Terminal::Settings::Model::Command> commandsCollection, winrt::TerminalApp::SuggestionsMode mode);
void _OpenSuggestions(const Microsoft::Terminal::Control::TermControl& sender, Windows::Foundation::Collections::IVector<winrt::Microsoft::Terminal::Settings::Model::Command> commandsCollection, winrt::TerminalApp::SuggestionsMode mode);

void _ShowWindowChangedHandler(const IInspectable sender, const winrt::Microsoft::Terminal::Control::ShowWindowArgs args);

Expand Down
1 change: 0 additions & 1 deletion src/cascadia/TerminalSettingsModel/ActionArgs.h
Original file line number Diff line number Diff line change
Expand Up @@ -816,7 +816,6 @@ namespace winrt::Microsoft::Terminal::Settings::Model::factory_implementation
BASIC_FACTORY(RenameTabArgs);
BASIC_FACTORY(SwapPaneArgs);
BASIC_FACTORY(SplitPaneArgs);
BASIC_FACTORY(SendInputArgs);
BASIC_FACTORY(SetFocusModeArgs);
BASIC_FACTORY(SetFullScreenArgs);
BASIC_FACTORY(SetMaximizedArgs);
Expand Down

0 comments on commit 902ee8e

Please sign in to comment.