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 SSH info to Instance Connect tab #2339

Merged
merged 11 commits into from
Aug 22, 2024
Merged

Add SSH info to Instance Connect tab #2339

merged 11 commits into from
Aug 22, 2024

Conversation

charliepark
Copy link
Contributor

@charliepark charliepark commented Jul 25, 2024

Fixes #2271

This adds an informational box to the Connect tab, talking about SSH and how the user can connect with their instance.

If they've set up any external IPs, it gives them most of what they need, using one of their floating IPs (and falling back to an ephemeral IP if no floating IPs).

image

If they haven't set up any external IPs, it directs them to the networking tab to set one up.

image

Learn more links go to the new Remote Access guide: https://docs.oxide.computer/guides/remote-access

Copy link

vercel bot commented Jul 25, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
console ✅ Ready (Inspect) Visit Preview Aug 22, 2024 7:37pm

@charliepark
Copy link
Contributor Author

We could maybe add one of the docs popup buttons to the top right of the SSH box to house the docs link, but I don't like how that'd add a whole new piece to that component.
Screenshot 2024-07-25 at 12 08 21 PM

@david-crespo
Copy link
Collaborator

I'm fine with the docs link right in the text.

Read our SSH guide to learn more.

or whatever

@paryhin
Copy link

paryhin commented Aug 5, 2024

This styling is slightly confusing, as the SSH information section looks like a modal with no buttons. For the information banners, we should stick to consistently using the blue style like this:

CleanShot 2024-08-05 at 13 25 22@2x

@david-crespo
Copy link
Collaborator

What about the serial console one? It also is not a modal. I was thinking these two should be styled the same because they are two ways of connecting to the instances. We might have a third one for VNC soon.

@paryhin
Copy link

paryhin commented Aug 5, 2024

These are different components: the first one is a card (e.g., Card Component), and the second one is an alert / banner / (e.g., Alert Component). They serve different purposes and have different properties. Cards are used for grouping related content and actions, while alerts are used to call out user attention and provide contextual information. For example, we may actually put the banner / alert in the card itself if we need a better visual grouping.

@david-crespo
Copy link
Collaborator

Current state:

image

<div className="flex gap-3">{children}</div>
</div>
),
Footer: classed.div`flex items-center justify-between border-t px-6 py-3 border-default h-14`,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Classic easier-to-compose-at-the-call-site situation

@david-crespo
Copy link
Collaborator

david-crespo commented Aug 20, 2024

This is close, but needs:

  • Copy once-over
  • Style check on inline code bit, it feels not quite right
  • A decent docs link for serial console

@charliepark
Copy link
Contributor Author

charliepark commented Aug 21, 2024

Current:
Screenshot 2024-08-21 at 4 56 32 PM

@david-crespo david-crespo enabled auto-merge (squash) August 22, 2024 19:38
@david-crespo david-crespo disabled auto-merge August 22, 2024 19:39
@david-crespo david-crespo enabled auto-merge (squash) August 22, 2024 19:40
@david-crespo david-crespo merged commit 1bb9270 into main Aug 22, 2024
8 checks passed
@david-crespo david-crespo deleted the ssh-info branch August 22, 2024 19:48
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.

Instance connect tab should talk about SSH
3 participants