shout icon indicating copy to clipboard operation
shout copied to clipboard

support for /j as an alias for /join

Open fczuardi opened this issue 10 years ago • 13 comments

Other IRC clients have by default a shorter version of the /join command, /j.

I think this could be a simple/nice to have enhancement.

fczuardi avatar Feb 23 '15 14:02 fczuardi

same with operator commands like /v = /voice

nwgat avatar Mar 26 '15 22:03 nwgat

and /l => /leave

docwhat avatar Apr 01 '15 17:04 docwhat

It should be /p -> /part.

There is no such thing as /LEAVE. That's probably an IRC client specific alias.

ghost avatar Apr 06 '15 18:04 ghost

Yeah, my bad. sigh.

docwhat avatar Apr 06 '15 19:04 docwhat

Yeah, I could add a couple of more shorthands. But you guys should try pressing TAB to complete commands. Like this /p<TAB>. You can even tabcomplete after just typing / to iterate all the available commands.

@ircer, @docwhat: I guess you're both right. Haha. LEAVE is not a valid IRC command. But I implemented both /part and /leave (and even /close) to close the current channel/query.

Code here for reference: https://github.com/erming/shout/blob/master/src/plugins/inputs/part.js

Feel free to dig into any of the files in src/plugins/inputs/ to add your own shorthands.

Also, I'll leave this here: http://shout-irc.com/docs/client/commands.html

Anyway, lets make a list of shorthands that would be nice to have:

/j, /join /p, /part /l, /leave /v, /voice /m, /msg

erming avatar Apr 06 '15 21:04 erming

We did something simular to this here: https://github.com/PonyChat/Shuo/pull/30

I only added aliases for the most used commands, as to prevent mistakes as well as the fact you can already type the first letter and hit tab.

floogulinc avatar Aug 08 '15 07:08 floogulinc

Anyway, lets make a list of shorthands that would be nice to have:

/q, /query

JocelynDelalande avatar Sep 27 '15 22:09 JocelynDelalande

It's slightly confusing to have no output when you use /j by mistake either, since you're not sure if it's just lagging.

Manouchehri avatar Dec 30 '15 21:12 Manouchehri

Please take a look at an earlier comment of mine regarding this issue. I am still :-1: on this considering the helpful tab completion.

astorije avatar Dec 31 '15 00:12 astorije

Is there any reason not to give an unknown command error though? I can't see the benefit of silently failing.

Manouchehri avatar Dec 31 '15 01:12 Manouchehri

Indeed, I believe failed commands should not be silent. However, I believe there was a good reason for this but I can't remember why. Does @JocelynDelalande or @erming remember?

If not, please feel free to open an issue and a PR for this, unless there are already some on the subject. Thanks!

astorije avatar Jan 06 '16 03:01 astorije

Indeed, I believe failed commands should not be silent. However, I believe there was a good reason for this but I can't remember why. Does @JocelynDelalande or @erming remember?

No good reason that I can see/remember, so PR welcome :)

JocelynDelalande avatar Jan 08 '16 13:01 JocelynDelalande

I guess you're both right. Haha. LEAVE is not a valid IRC command. But I implemented both /part and /leave (and even /close) to close the current channel/query.

The /close command is a ircops command for at least some servers (such as unrealircd).

This is related to what I've said in 528. It might be best to only handle commands that you want to alter specific behavior for (such as /part working without including the current channel name).

hormigas avatar Apr 13 '16 00:04 hormigas