Skip to content
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

Add a command palette launcher to the title bar #149445

Closed
jrieken opened this issue May 13, 2022 · 17 comments
Closed

Add a command palette launcher to the title bar #149445

jrieken opened this issue May 13, 2022 · 17 comments
Assignees
Labels
command-center feature-request Request for new features or functionality verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@jrieken
Copy link
Member

jrieken commented May 13, 2022

The quick pick family, command palette, search etc, is very powerful but often overlooked. Esp new users don't know about its keybindings are rarely discover it. The idea is to render a "launcher" in the title bar part that simply triggers quick pick, like so

Screen.Recording.2022-05-13.at.12.12.22.mov

This is related to #115641

@jrieken jrieken self-assigned this May 13, 2022
@jrieken jrieken added this to the May 2022 milestone May 13, 2022
@jrieken jrieken added feature-request Request for new features or functionality command-center labels May 13, 2022
@jrieken
Copy link
Member Author

jrieken commented May 13, 2022

explore/experiment for May, configuration option for insiders: window.experimental.titleMenu

@jrieken
Copy link
Member Author

jrieken commented May 13, 2022

/cc @daviddossett @misolori

@jrieken
Copy link
Member Author

jrieken commented May 13, 2022

I guess we should also consider to update the height of the title bar part, right? This is how it could look with some extra height

Screenshot 2022-05-13 at 12 41 28

@jrieken
Copy link
Member Author

jrieken commented May 13, 2022

With #149468 I have added support for custom hover here. @sbatten if this sticks we should also do the same for the layout switcher

@jrieken
Copy link
Member Author

jrieken commented May 13, 2022

Screen.Recording.2022-05-13.at.15.28.03.mov

@daviddossett
Copy link
Contributor

daviddossett commented May 13, 2022

I guess we should also consider to update the height of the title bar part, right? This is how it could look with some extra height

How would this look on Windows or other platforms w/ the classic menu? Would those be centered vertically?

CleanShot 2022-05-13 at 08 51 25@2x

Overall I think it's a good idea. Things feel quite tight as is—although some will prefer that. It could be interesting to think about different height settings if this setting is enabled. However that could be a bit tricky with the existing menu bar style setting being an added dimension here.

Are we able to align the window actions vertically on macOS? Ours are still aligned to the top of the container:

CleanShot 2022-05-13 at 08 49 00@2x

Here's how they look centered in some other native/electron apps:

Safari

CleanShot 2022-05-13 at 09 25 46@2x

Slack

CleanShot 2022-05-13 at 09 26 03@2x

Outlook

CleanShot 2022-05-13 at 09 26 19@2x

@jrieken
Copy link
Member Author

jrieken commented May 16, 2022

Are we able to align the window actions vertically on macOS? Ours are still aligned to the top of the container:

I would hope so but idk where that currently comes from...

@sbatten
Copy link
Member

sbatten commented May 16, 2022

@jrieken we can control that with trafficLightPosition on BrowserWindow https://www.electronjs.org/docs/latest/api/browser-window

if (useCustomTitleStyle) {
options.titleBarStyle = 'hidden';
this.hiddenTitleBarStyle = true;
if (!isMacintosh) {
options.frame = false;
}

@imthenachoman
Copy link

Reading this thread, I'm a bit confused. Was this option added? I can't find it in the settings but I see it in y'all's screenshots?

@jrieken
Copy link
Member Author

jrieken commented May 23, 2022

#149445 (comment)

@imthenachoman
Copy link

I see. I will download insider build. Thank you.

One feedback, window.experimental.titleMenu is a bit ambiguous and confusing. window.experimental.titleMenuSearch orr something would be a more obvious setting.

@jrieken
Copy link
Member Author

jrieken commented May 23, 2022

Yeah, we haven't decided how to name this thing yet. The current favourite is "command center"

@imthenachoman
Copy link

Sometimes I spend more time figuring out how to name my variables/settings then actually writing the code. It's such an existential philosophical challenge. Good luck! Love this feature. Thanks!

@jrieken jrieken mentioned this issue May 25, 2022
3 tasks
@heartacker
Copy link
Contributor

now we could click the textbox to launch global search.
and we could click the combo box style to get more action,
I suggest adding another button to launching the command panel directly

image

@tjx666

This comment was marked as off-topic.

@jrieken
Copy link
Member Author

jrieken commented Jun 27, 2022

Closing as the main part of the feature has been implemented. More feedback is very welcome but should be filed as new issues. Issues in this feature-area a label command-center

@jrieken jrieken closed this as completed Jun 27, 2022
@jrieken
Copy link
Member Author

jrieken commented Jun 27, 2022

To verify:

  • ensure the window.commandCenter-setting toggles the command center

@jrieken jrieken added the verification-needed Verification of issue is requested label Jun 27, 2022
@alexdima alexdima added verified Verification succeeded and removed verification-needed Verification of issue is requested labels Jun 28, 2022
@rzhao271 rzhao271 added the verification-needed Verification of issue is requested label Jun 28, 2022
@miguelsolorio miguelsolorio mentioned this issue Aug 4, 2022
52 tasks
@github-actions github-actions bot locked and limited conversation to collaborators Aug 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
command-center feature-request Request for new features or functionality verification-needed Verification of issue is requested verified Verification succeeded
Projects
Status: 💻 In Development
Development

No branches or pull requests

9 participants