Optimize the display of reasons #612

New issue

Open

opened 2023-12-29 05:05:19 +13:00 by dtonon · 3 comments

dtonon commented

2023-12-29 05:05:19 +13:00

(Migrated from github.com)

We have some problems with the visualization of the reasons in the relays view; the space is not enough and the text overlap the main view and the sidebar.

A first quick solution could be to move them below the stats row, and show them only when the details option is enabled. This way they will have a full row width.

An alternative could be use icons, but it is not so easy to visually explain all the actions.

@mikedilger @bu5hm4nn what do you think?

We have some problems with the visualization of the reasons in the relays view; the space is not enough and the text overlap the main view and the sidebar. A first quick solution could be to move them below the stats row, and show them only when the details option is enabled. This way they will have a full row width. An alternative could be use icons, but it is not so easy to visually explain all the actions. @mikedilger @bu5hm4nn what do you think? <img width="917" alt="image" src="https://github.com/mikedilger/gossip/assets/89577423/fbbdbd25-e136-4f18-9243-f29953da7303">

mikedilger commented

2023-12-29 22:28:45 +13:00

(Migrated from github.com)

This is a bug. There should not be more than one "FetchMentions" happening on a relay.

in apply_relay_assignment() I guess I need to check if FetchMentions is already on that relay.

This is a bug. There should not be more than one "FetchMentions" happening on a relay. in apply_relay_assignment() I guess I need to check if FetchMentions is already on that relay.

mikedilger commented

2024-04-20 13:21:50 +12:00

(Migrated from github.com)

is this fixed?

bu5hm4nn commented

2024-04-22 09:06:21 +12:00

(Migrated from github.com)

They are now only shown when Debug statistics are enabled. Not a full fix, but fixed for the average user