Skip to content

Conversation

@lzl1918
Copy link
Contributor

@lzl1918 lzl1918 commented Jun 26, 2021

close #490

@taooceros
Copy link
Member

Thanks for the fix! Would you please take a check whether we need to call Activate()? Previously testing shows that calling Focus() is enough.

@taooceros
Copy link
Member

By the way, would you please also add the Focus() or Activate() here?

SetHotkey(new HotkeyModel(updateCustomHotkey.Hotkey), delegate
{
App.API.ChangeQuery(updateCustomHotkey.ActionKeyword);
Application.Current.MainWindow.Visibility = Visibility.Visible;
});

@lzl1918
Copy link
Contributor Author

lzl1918 commented Jun 26, 2021

By the way, would you please also add the Focus() or Activate() here?

SetHotkey(new HotkeyModel(updateCustomHotkey.Hotkey), delegate
{
App.API.ChangeQuery(updateCustomHotkey.ActionKeyword);
Application.Current.MainWindow.Visibility = Visibility.Visible;
});

Thanks. Could you please help check and review? @taooceros

@taooceros
Copy link
Member

Thank you! I am afk today. Will review it tomorrow!

@taooceros
Copy link
Member

Thanks for the fix!

@taooceros taooceros merged commit a312fe8 into Flow-Launcher:dev Jun 27, 2021
@jjw24 jjw24 added the bug Something isn't working label Jun 27, 2021
This was referenced Jun 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Plugin.Shell: The query box is not auto-focused when its hotkey is invoked

3 participants