-
-
Notifications
You must be signed in to change notification settings - Fork 45
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 more CLI flags #186
Comments
It's more correct to name it |
I'm thinking out loud
Another observation is that an empty workspace might be the "subject" as well. The workspace flag needs to be named with a symmetrical name Proposals:
Description proposal: Make the command acts as if the Other things that need to be thought about are:
|
How about |
(1) Add
--window-id
option to all or almost all commands. This option would make the command behave like if<window-id>
was focusedMotivation:
--window-id
is already presented. That's howon-window-detected.run
is implemented. We could make it more transparent by saying in the docs that--window-id
is applied by default to all of theon-window-detected.run
commands. Lift limitations inon-window-detected.run
#20focus
command then we force focus the current window. This way focusing the window by id becomesaerospace --window-id <ID> focus
(2) Add
--no-follow-focus
tomove
and--follow-focus
tomove-node-to-workspace
commands (btw, it's inconsistent thatmove
follows focus by default, butmove-node-to-workspace
doesn't).Motivation:
aerospace move-node-to-workspace <workspace> && aerospace workspace <workspace>
, but it gets more problematic whenmove-node-to-workspace
is invoked byxargs
. "Follow focus" is quite a common use case, it would be nice to have a flag for it.on-window-detected.run
#20 by saying that--follow-focus
is applied by default tomove-node-to-workspace
(3) Add
--format window-id
(or--window-id-only
) tolist-windows
command.Motivation: It's tedious and error-prone to write
aerospace list-windows | awk '{print $1}'
.In the same way,
--format monitor-id
(or--monitor-id-only
) can be added tolist-monitors
commandMotivation for all 3 flags:
(a) Move all windows to the current workspace #185:
The text was updated successfully, but these errors were encountered: