RTL icon indicating copy to clipboard operation
RTL copied to clipboard

default channels per page & sort options

Open seth586 opened this issue 4 years ago • 10 comments

Every time I use RTL I am clicking "channels per page: 100" and then clicking "peer" to sort alphabetically.

It would be nice to reduce this workflow for my routing node maintenance routine by giving us options to set defaults in RTL-Config.json such as (example):

"channelsperpage": "100",
"peersperpage": "100",
"routingSort": "timestamp",

etc....

seth586 avatar Mar 29 '20 04:03 seth586

@saubyk As discussed earlier, we were not in favor of adding these many configurations to RTL. But, it looks like a very popular feature request.

Do you want to reconsider your decision? If yes, we need to discuss the UX/UI for the feature.

ShahanaFarooqui avatar Jun 09 '20 16:06 ShahanaFarooqui

@ShahanaFarooqui we can consider creating a small database for config options. It can help us scale other capabilities as well on the application.

saubyk avatar Jun 11 '20 01:06 saubyk

Also sorting by the peer in the list is not valid, CASE letters are taking precedent in the alias name sort and is not sorted Alpha correctly. i.e. an alias with uppercase B is sorted at top and lower case b is further below in list.

mjlamb avatar Jul 31 '20 10:07 mjlamb

The latest release has resolved the "peer" column sorting, but sats column sorting is still broken.... does not sort sats correctly.

1,043,390 1,081,790 114,665 119,643

etc

mjlamb avatar Aug 03 '20 07:08 mjlamb

@mjlamb Though I was unable to reproduce 'Sats' sorting issue on my local, I hope that the latest commit at 0.8.4 branch will fix it from v.0.8.4.

ShahanaFarooqui avatar Aug 03 '20 12:08 ShahanaFarooqui

would you consider adding all so it's possible to show (and search) all channels/peers in one go, not just the first 100?

viaj3ro avatar Aug 19 '20 09:08 viaj3ro

Should default to showing 100 or All, which is the smallest.

mjlamb avatar Aug 19 '20 10:08 mjlamb

yeah, but once you are above 100 channels, there is now way to show them all on one page at the moment. So adding the option to show all, no matter what, would do the trick.

viaj3ro avatar Aug 19 '20 10:08 viaj3ro

Only able to show 100? That is a strange constraint.

mjlamb avatar Aug 19 '20 10:08 mjlamb

I would also like an option to list all channels at once instead of just max 100.

also PLEASE add the "channels per page" dropdown to the TOP of the page as well. currently I have to scroll all the way down, change it to 100, then scroll all the way back up as it puts you at the bottom of the page after setting it to 100. Its very annoying. Adding the dropdown to the top of the page as well would massively improve the UX for when you are looking at channels and need to sort them in various ways.

sendbitcoin avatar Dec 05 '20 14:12 sendbitcoin

Closing it with PR #1127. Released with v0.13.2.

ShahanaFarooqui avatar Nov 22 '22 17:11 ShahanaFarooqui