erming/shout

[feature request] listing channels

scotticles opened this issue · 8 comments

would be nice to have a /list command.

Thanks.

https://tools.ietf.org/html/rfc2812#section-3.2.6 IRC spec for /list command in case someone wants to implement it

Hmm, interesting. I always assumed /list accepted arguments such as /list >500 to retrieve channels with more than 500 users. But according to that document this is not part of the spec at all.
This would require some changes in the slate/slate-irc library we use though, as far as i can tell it's not implemented there yet.

Also, would you prefer to see this in some new window, or should we just return this in the network channel, like the MOTD? That last option would be easier to implement, but definitely not as user friendly.

I'd personally like to see stuff like this (and the /whois information as mentioned in 214) in a modal window kinda like how IRCCloud does it. But that would make it a much bigger story, and I know @astorije will jump in with some UX advice (or at least, saying we need to get some).

@dubzi I think it could be implemented shout-side, but I can't think of a reason not to make an upstream PR.

What do we think about protecting our users with potentially destructive commands? I was testing this myself, and ran a /list on freenode. Without any arguments, that command works, and is valid, but will return every channel on freenode. It crashed my browser 😨

IMO we just shouldn't accept that command without any arguments. Perhaps returning an example of arguments you can use when you run it without arguments?

I also found an answer to my question earlier about the spec. Basically the custom arguments such as /list >500 are handled on the irc server's side. Not sure which servers accept it since it's not part of the spec, but I assume most do.

This issue was moved to thelounge/thelounge#130

Didn't mean to close this, sorry, https://github-issue-mover.appspot.com/ did it automatically, duh...

@dubzi Other clients solve this with "Are you sure you want to do that? Run /list with --yes" when you just enter /list by itself.

That said, it would be really awesome if this client were FULLY rfc2812 compliant. And if server messages would show in the server "tab". :)