Skip to content

fix: simplify terminal link #2597

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

Merged
merged 2 commits into from
Jun 22, 2022
Merged

fix: simplify terminal link #2597

merged 2 commits into from
Jun 22, 2022

Conversation

f0ssel
Copy link
Contributor

@f0ssel f0ssel commented Jun 22, 2022

This is just a proposal - I think next to the code-server link it looks better.

Currently -

image

"terminal"

image

"web-terminal" (what we commonly call it internally)

image

"ssh" - this is what GCP uses for their web terminal

image

We landed on "Terminal"

image

@f0ssel f0ssel requested a review from a team as a code owner June 22, 2022 21:24
@f0ssel
Copy link
Contributor Author

f0ssel commented Jun 22, 2022

@Kira-Pilot I added some options for alternative text, could you comment on what your preference is?

Anyone else's input is also very welcome.

Copy link
Member

@kylecarbs kylecarbs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd still capitalize the T, but I agree with removing "Open". It's redundant for links!

@Kira-Pilot
Copy link
Member

@f0ssel I think Terminal with a capital T makes the most sense to me!

@ntimo
Copy link
Contributor

ntimo commented Jun 22, 2022

I personally think just calling it terminal is totally sufficient. Adding different versions per cloud could also be confusing and is probably overkill in my opinion.

@f0ssel f0ssel merged commit 6a2a145 into main Jun 22, 2022
@f0ssel f0ssel deleted the f0ssel/simple-terminal branch June 22, 2022 22:11
@ketang
Copy link
Contributor

ketang commented Jun 23, 2022

Customer perspective should win. We should either go with that @ntimo said or ask more customers, but an internal opinion shouldn't overrule a customer opinion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants