Review Relays section #834
Labels
No labels
Blocked
Bug
Documentation
Duplicate
Enhancement
Good first issue
Help wanted
Idea
In progress
Invalid
Major feature set
Packaging
Question
Soon
UI/UX
Upstream
You're dreamin'
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
nostr/gossip#834
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Here are some fixes to clean up and improve (imo) the Relay section:
General interface
Lists
Relay details
I skipped moving the "Advertise Relays" button for now, because it is also used as a status indicator for the advertisement (it displays "n relays to go"). I think we need a general solution for displaying status of "background tasks".
Is the realtime update really important for the user? I don't think so.
I would keep trace only of the final status, and for this we need to enable the general log view. But it's an optional step.
Yes I agree. People don't want to look at terminal logs, but they do want to know if their posts succeeded to the relays they are going to, and when advertise is done, etc.
@bu5hm4nn there is a scrolling issue with long relay names:
I also agree, and we actually have a free slot on the bottom of the sidebar for this.
We can show both informative logs (maybe the level can be set by a preference) and the in-progress tasks, putting them on the top with a different style (e.g. a light animation).
But for this specific case, I think that, for now, we can safely avoid showing the user a counter that goes from 250+ to zero.
Should be all done on branch
Looks good to me, except that the relay path is missing. If relays have a path (which is the exception) it must be shown. I'm fine with
wss://being elided, as well as trailing slashes. But we need the path. I insist.Gossip picks up a lot of bad relay records. I will try to work on pruning those out. For the moment, you can't identify the bad ones without the path.
@mikedilger
I agree.
To have a cleaner default view, we can show it (together with the port) when the relay is expanded, like this:
And in the details mode:
What do you think?
@bu5hm4nn almost perfect, just this notes:
I would further reduce the search width, 8-9 characters seems sufficient to filter the relay list. This way we have more space to show the title when the window is compact.
The tooltips are still present.
The outbox preview box is not always clickable, it have blind spots, you can verify it from the fact that the mouse cursor changes when hovering.
Do we have a neutral (gray) version of the dropdown button? I would prefer it.
I added an item:
That's ok. As long as it is there.
@bu5hm4nn can you that the path/port as proposed?
Please re-test the branch. I am not confident enough to remove the tooltip with the full url string, I might have missed some cases (maybe some with arguments).