mirror of
https://github.com/microsoft/PowerToys.git
synced 2026-04-04 18:26:39 +02:00
Add the Command Palette module (#37908)
Windows Command Palette ("CmdPal") is the next iteration of PowerToys Run. With extensibility at its core, the Command Palette is your one-stop launcher to start _anything_.
By default, CmdPal is bound to <kbd>Win+Alt+Space</kbd>.
----
This brings the current preview version of CmdPal into the upstream PowerToys repo. There are still lots of bugs to work out, but it's reached the state we're ready to start sharing it with the world. From here, we can further collaborate with the community on the features that are important, and ensuring that we've got a most robust API to enable developers to build whatever extensions they want.
Most of the built-in PT Run modules have already been ported to CmdPal's extension API. Those include:
* Installed apps
* Shell commands
* File search (powered by the indexer)
* Windows Registry search
* Web search
* Windows Terminal Profiles
* Windows Services
* Windows settings
There are a couple new extensions built-in
* You can now search for packages on `winget` and install them right from the palette. This also powers searching for extensions for the palette
* The calculator has an entirely new implementation. This is currently less feature complete than the original PT Run one - we're looking forward to updating it to be more complete for future ingestion in Windows
* "Bookmarks" allow you to save shortcuts to files, folders, and webpages as top-level commands in the palette.
We've got a bunch of other samples too, in this repo and elsewhere
### PowerToys specific notes
CmdPal will eventually graduate out of PowerToys to live as its own application, which is why it's implemented just a little differently than most other modules. Enabling CmdPal will install its `msix` package.
The CI was minorly changed to support CmdPal version numbers independent of PowerToys itself. It doesn't make sense for us to start CmdPal at v0.90, and in the future, we want to be able to rev CmdPal independently of PT itself.
Closes #3200, closes #3600, closes #7770, closes #34273, closes #36471, closes #20976, closes #14495
-----
TODOs et al
**Blocking:**
- [ ] Images and descriptions in Settings and OOBE need to be properly defined, as mentioned before
- [ ] Niels is on it
- [x] Doesn't start properly from PowerToys unless the fix PR is merged.
- https://github.com/zadjii-msft/PowerToys/pull/556 merged
- [x] I seem to lose focus a lot when I press on some limits, like between the search bar and the results.
- This is https://github.com/zadjii-msft/PowerToys/issues/427
- [x] Turned off an extension like Calculator and it was still working.
- Need to get rid of that toggle, it doesn't do anything currently
- [x] `ListViewModel.<FetchItems>` crash
- Pretty confident that was fixed in https://github.com/zadjii-msft/PowerToys/pull/553
**Not blocking / improvements:**
- Show the shortcut through settings, as mentioned before, or create a button that would open CmdPalette settings.
- When PowerToys starts, CmdPalette is always shown if enabled. That's weird when just starting PowerToys/ logging in to the computer with PowerToys auto-start activated. I think this should at least be a setting.
- Needing to double press a result for it to do the default action seems quirky. If one is already selected, I think just pressing should be enough for it to do the action.
- This is currently a setting, though we're thinking of changing the setting even more: https://github.com/zadjii-msft/PowerToys/issues/392
- There's no URI extension. Was surprised when typing a URL that it only proposed a web search.
- [x] There's no System commands extension. Was expecting to be able to quickly restart the computer by typing restart but it wasn't there.
- This is in PR https://github.com/zadjii-msft/PowerToys/pull/452
---------
Co-authored-by: joadoumie <98557455+joadoumie@users.noreply.github.com>
Co-authored-by: Jordi Adoumie <jordiadoumie@microsoft.com>
Co-authored-by: Mike Griese <zadjii@gmail.com>
Co-authored-by: Niels Laute <niels.laute@live.nl>
Co-authored-by: Michael Hawker <24302614+michael-hawker@users.noreply.github.com>
Co-authored-by: Stefan Markovic <57057282+stefansjfw@users.noreply.github.com>
Co-authored-by: Seraphima <zykovas91@gmail.com>
Co-authored-by: Jaime Bernardo <jaime@janeasystems.com>
Co-authored-by: Kristen Schau <47155823+krschau@users.noreply.github.com>
Co-authored-by: Eric Johnson <ericjohnson327@gmail.com>
Co-authored-by: Ethan Fang <ethanfang@microsoft.com>
Co-authored-by: Yu Leng (from Dev Box) <yuleng@microsoft.com>
Co-authored-by: Clint Rutkas <clint@rutkas.com>
This commit is contained in:
Mike Griese
108
src/modules/cmdpal/Microsoft.Terminal.UI/Converters.cpp
Normal file
108
src/modules/cmdpal/Microsoft.Terminal.UI/Converters.cpp
Normal file
@@ -0,0 +1,108 @@
|
||||
#include "pch.h"
|
||||
#include "Converters.h"
|
||||
#include "Converters.g.cpp"
|
||||
|
||||
#pragma warning(disable : 26497) // We will make these functions constexpr, as they are part of an ABI boundary.
|
||||
#pragma warning(disable : 26440) // The function ... can be declared as noexcept.
|
||||
|
||||
namespace winrt::Microsoft::Terminal::UI::implementation
|
||||
{
|
||||
// Booleans
|
||||
bool Converters::InvertBoolean(bool value)
|
||||
{
|
||||
return !value;
|
||||
}
|
||||
|
||||
// winrt::Windows::UI::Xaml::Visibility Converters::InvertedBooleanToVisibility(bool value)
|
||||
// {
|
||||
// return value ? winrt::Windows::UI::Xaml::Visibility::Collapsed : winrt::Windows::UI::Xaml::Visibility::Visible;
|
||||
// }
|
||||
|
||||
// Numbers
|
||||
double Converters::PercentageToPercentageValue(double value)
|
||||
{
|
||||
return value * 100.0;
|
||||
}
|
||||
|
||||
double Converters::PercentageValueToPercentage(double value)
|
||||
{
|
||||
return value / 100.0;
|
||||
}
|
||||
|
||||
//winrt::hstring Converters::PercentageToPercentageString(double value)
|
||||
//{
|
||||
// return winrt::hstring{ fmt::format(FMT_COMPILE(L"{:.0f}%"), value * 100.0) };
|
||||
//}
|
||||
|
||||
// Strings
|
||||
bool Converters::StringsAreNotEqual(const winrt::hstring& expected, const winrt::hstring& actual)
|
||||
{
|
||||
return expected != actual;
|
||||
}
|
||||
|
||||
bool Converters::StringNotEmpty(const winrt::hstring& value)
|
||||
{
|
||||
return !value.empty();
|
||||
}
|
||||
|
||||
// winrt::Windows::UI::Xaml::Visibility Converters::StringNotEmptyToVisibility(const winrt::hstring& value)
|
||||
// {
|
||||
// return value.empty() ? winrt::Windows::UI::Xaml::Visibility::Collapsed : winrt::Windows::UI::Xaml::Visibility::Visible;
|
||||
// }
|
||||
|
||||
winrt::hstring Converters::StringOrEmptyIfPlaceholder(const winrt::hstring& placeholder, const winrt::hstring& value)
|
||||
{
|
||||
return placeholder == value ? L"" : value;
|
||||
}
|
||||
|
||||
// Misc
|
||||
// winrt::Windows::UI::Text::FontWeight Converters::DoubleToFontWeight(double value)
|
||||
// {
|
||||
// return winrt::Windows::UI::Text::FontWeight{ base::ClampedNumeric<uint16_t>(value) };
|
||||
// }
|
||||
|
||||
//winrt::Windows::UI::Xaml::Media::SolidColorBrush Converters::ColorToBrush(const winrt::Windows::UI::Color color)
|
||||
//{
|
||||
// return Windows::UI::Xaml::Media::SolidColorBrush(color);
|
||||
//}
|
||||
|
||||
// double Converters::FontWeightToDouble(const winrt::Windows::UI::Text::FontWeight fontWeight)
|
||||
// {
|
||||
// return fontWeight.Weight;
|
||||
// }
|
||||
|
||||
// double Converters::MaxValueFromPaddingString(const winrt::hstring& paddingString)
|
||||
// {
|
||||
// std::wstring_view remaining{ paddingString };
|
||||
// double maxVal = 0;
|
||||
|
||||
// // Get padding values till we run out of delimiter separated values in the stream
|
||||
// // Non-numeral values detected will default to 0
|
||||
// // std::stod will throw invalid_argument exception if the input is an invalid double value
|
||||
// // std::stod will throw out_of_range exception if the input value is more than DBL_MAX
|
||||
// try
|
||||
// {
|
||||
// while (!remaining.empty())
|
||||
// {
|
||||
// const std::wstring token{ til::prefix_split(remaining, L',') };
|
||||
// // std::stod internally calls wcstod which handles whitespace prefix (which is ignored)
|
||||
// // & stops the scan when first char outside the range of radix is encountered
|
||||
// // We'll be permissive till the extent that stod function allows us to be by default
|
||||
// // Ex. a value like 100.3#535w2 will be read as 100.3, but ;df25 will fail
|
||||
// const auto curVal = std::stod(token);
|
||||
// if (curVal > maxVal)
|
||||
// {
|
||||
// maxVal = curVal;
|
||||
// }
|
||||
// }
|
||||
// }
|
||||
// catch (...)
|
||||
// {
|
||||
// // If something goes wrong, even if due to a single bad padding value, we'll return default 0 padding
|
||||
// maxVal = 0;
|
||||
// LOG_CAUGHT_EXCEPTION();
|
||||
// }
|
||||
|
||||
// return maxVal;
|
||||
// }
|
||||
}
|
||||
Reference in New Issue
Block a user