feat: add connection status, disconnect button, and recent tcp connections list#526
Conversation
|
WOW, this is an awesome PR. This is something people have wanted for a while. I'm looking forward to merging this once I do some testing. I've added a couple comments for things to discuss as well. Could you try rebasing on the |
|
@matthewCmatt thanks for merging the bluetooth changes. The only real conflict was that I removed I updated the disconnect icon and moved the connection status card as you suggested. I think it looks a lot better. I updated the screenshots in the original PR |
|
closes #458 |
|
Tested manually on MacOS using bluetooth and serial connection. Disconnection on bluetooth seems a little buggy, but that's probably something with Thanks @itslenny! |
Background
I recently discovered this project, and am super excited to use (and contributing to) it extensively. The first issue I ran into is that it is a bit of a pain to switch between my nodes, and I kept forgetting which node I was connected to. So, this PR adds a connection status card, a disconnect button, and a list of recent connections. These features allows me to quickly switch between my nodes, and helps me not forget which one I'm connected to.
Changes
Screenshots
When there are no recent connections (when user upgrades to this version) the UI appears unchanged

Once you connect you'll see a connection status card with a disconnect button

Disconnect button with sidebar collapsed

Connection status (serial connection)

Connection status (bluetooth)

Recent connections are shown in a list below the existing TCP connection form

New connecting screen with cancel button
