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

Feature Request: Add attributes to [channels] header #67

Open
dekdynamics opened this issue Apr 13, 2022 · 3 comments
Open

Feature Request: Add attributes to [channels] header #67

dekdynamics opened this issue Apr 13, 2022 · 3 comments

Comments

@dekdynamics
Copy link

dekdynamics commented Apr 13, 2022

Hello,

The current header bar in lntop looks like this:

 The Wall lnd-v0.14.2 bitcoin mainnet [synced] height: 731695 peers: 153 

 [ Channels ]                                                                  [ Wallet ]
 balance:                                                                balance:   
 state  : 150 active 0 pending 1 inactive
 gauge  : [|||||||||||         ] 58%

Would it be possible to add some attributes here?

  1. Change "inactive" to "disconnected" for clarity
  2. Add "Disabled remote" to indicate which peers have disabled a channel
  3. Add "Disabled local" to indicate which channels are locally disabled

Thanks,
DEK

@dekdynamics dekdynamics changed the title Feature Request: Add attributes to [channels Feature Request: Add attributes to [channels] header Apr 13, 2022
@hieblmi
Copy link
Contributor

hieblmi commented Jul 30, 2022

I support active/inactive to connected/disconnected!

@hieblmi
Copy link
Contributor

hieblmi commented Sep 23, 2022

for 1. @rkfg to weigh in - Question is should we rename active to connected and inactive to disconnected.
2. and 3. have been fixed here: https://github.com/edouardparis/lntop/pull/76/files#diff-1a61b9279a10d79e5322cbbc01644da76c7b9e7111de5f718d6fe7a0bc83ec68R665

@rkfg
Copy link
Contributor

rkfg commented Sep 23, 2022

Agree that connected/disconnected make much more sense. However, the words themselves are too long for this column and disconnected ⇈ wouldn't even fit. Maybe simple on/off would do?

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

No branches or pull requests

3 participants