-
-
Notifications
You must be signed in to change notification settings - Fork 131
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
✨ Adding support for windows terminal #39
✨ Adding support for windows terminal #39
Conversation
bumped "selectionBackground" slightly up because it was not visible. text color has been assigned to "foreground"
Windows powershell uses brightblack as command line switches for commands.
The link on the main Readme.md will navigate to the windows terminal heading correctly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for submitting a PR! 👀
@@ -0,0 +1,23 @@ | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey! Since the folder is already named windows
I think we should rename windowsterminal
to terminal
?
Wdyt 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that's a good catch, thank you!
renaming windowsterminal to terminal because the folder one level up is also named "windows". path windows/terminal is better then windows/windowsterminal.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 🚀
Hello there,
I saw there is no support for Windows Terminal. Hence I traced back my own steps and I am submitting a pull request!
Here are some things to consider,
#9cb0b9
Thank you!