-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
Discussion: commandline arguments in v0.9 are not user-friendly or readable? #4601
Comments
I have not tested it but I think that should work: wt --profile "Windows PowerShell" --startingDirectory . ; split-pane -V wsl.exe`
wt --startingDirectory C:\Users\cinnamon\GitHub\WindowsTerminal ; split-pane --profile "Command Prompt" ; split-pane --profile "Ubuntu" --startingDirectory \wsl$\Ubuntu\home\cinnak -H You can read the doc here: https://github.com/microsoft/terminal/blob/master/doc/user-docs/UsingCommandlineArguments.md |
Yes majorly this is ok for some reason it uses -- instead of - which is uncommon to Windows given my suggestion it should be more understood like autocomplete or readable args let's compare with diskpart where you can say lis dis so for terminal this should be -p or -profile not only --profile to make it easier to distinguish. so either as we are still in the beginning of args in WT I hope design decisions can still be made and yet we are in a beta, no one can complain about more drastical changes here (there should be few productive scripts) |
another addition about consistency: If we want to keep -- args for linux friendlyness, please allow to additionally use all args with just one - for Windows / Powershellists This might please all usergroups which are used to - or -- arguments. |
Well, I don't think Windows-native command-line apps have ever had a consistent (or readable) parameter experience, outside of PowerShell 😉 But why are we using linux syntax Before PowerShell, Windows apps did not use dashes at all, never mind double-dashes. We used In PowerShell, parameters are not case sensitive, and we still don't support the concept of combining short-form parameters. The syntax |
Perhaps it could be made better but you are forgetting the primary users of Terminal are also using WSL. It is extremely likely that compatibility will be broken if that syntax isn't used. |
I'm marking this as the discussion area for commandline arguments. |
I don't know what makes those of us using WSL the "primary" users. Every user of Terminal is also using Windows. WSL is a Windows app (which is why it has awful pop-up help behavior).
I'm sorry, but that's hogwash. Show me how using The idea that we should use posix-normal parameter syntax in a Windows app that pops up dialogs to show help is really just guaranteeing confusion. WSL users will use a lot of windows apps, both CLI and Windowed. None of those apps will use this syntax. |
@DHowett-MSFT the original intention is a feature request. I am unsure if "discussions" will change anything in the product. Can you outline how this discussion will be handled by PM and designers? |
Okay so that's on me for not adding the long form of those args. I kinda threw the whole In general, we'll be having both long and short forms for the args we add to commands. Everything else in that spec has them, but sometimes when writing/reviewing a 700 line spec, you miss some of the details 😜 Just now, I actually discovered that CLI11 (which we're using for parsing the args) actually also implicitly supports the |
I am looking forward how this will turn out in the future. |
Hey so this discussion seems to have run it's course. Since we're getting imminently close to 1.0 at this point, and it doesn't seem like we're about to dramatically change the surface of the commandline args, I'm going to close this discussion. I think we've addressed most of the complaints in the OP about not having Thanks for the feedback everyone! |
Make arguments user-friendly and type friendly
With ver 0.9 arguments to launch WT has been introduced. This is really a good feature addition.
However I find that the arguments are very linux like and unfriendly to new users:
please refer Ned Pyle:
https://twitter.com/NerdPyle/status/1227757778732601344
and Cinnamon examples:
wt -p "Windows PowerShell" -d . ; split-pane -V wsl.exe
wt -d C:\Users\cinnamon\GitHub\WindowsTerminal ; split-pane -p "Command Prompt" ; split-pane -p "Ubuntu" -d \wsl$\Ubuntu\home\cinnak -H
I do not feel this is userfriendly. Not everyone that want to replace cmd and powershell with a modern terminal is a dev
implement arguments that allow both written and readable and shortened args example, which exlcude using ambigous args
examples
-r or -run
-l or -list
The text was updated successfully, but these errors were encountered: