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

Show hostmask on join/part/quit #55

Closed
v0l opened this issue Feb 16, 2016 · 11 comments
Closed

Show hostmask on join/part/quit #55

v0l opened this issue Feb 16, 2016 · 11 comments
Labels
Type: Feature Tickets that describe a desired feature or PRs that add them to the project.
Milestone

Comments

@v0l
Copy link

v0l commented Feb 16, 2016

Add host info for people who join a chan,

v0l (v0@ip) has joined

Also add quit messages

@xPaw
Copy link
Member

xPaw commented Feb 16, 2016

Quit messages are already shown, do you not see them?

@v0l
Copy link
Author

v0l commented Feb 16, 2016

Oh yea i see it, needed update sorry.

@v0l v0l changed the title Show extra info on join & leave Show extra info on join Feb 16, 2016
@xPaw xPaw changed the title Show extra info on join Show hostmask on join/part/quit Feb 16, 2016
@xPaw xPaw added the Type: Feature Tickets that describe a desired feature or PRs that add them to the project. label Feb 16, 2016
@astorije
Copy link
Member

I really don't think we shout display host info when someone joins. It's noisy, doesn't add much information, and can be retrieved using a /query. I find that negatively affecting the UX so I'm 👎 on this, sorry!

@dgw
Copy link
Contributor

dgw commented Feb 17, 2016

Gonna go out on a limb here and say that themes should be able to add or remove info in event messages, regardless of whether core does this. (If that's not currently possible/planned, please do correct me!)

@astorije You mean a /whois, no?

@astorije
Copy link
Member

Sorry yes, a /whois. And I disagree with this, themes should be about UI, not data/info. Anything else too specific and not UI-related should go in a package.

@xPaw
Copy link
Member

xPaw commented Feb 17, 2016

It's noisy, doesn't add much information, and can be retrieved using a /query.

It can't be retrieved using commands when you're trying to ban someone that just quit. We should have a way to display hostmasks (by having an option in settings, if so desire), in these messages.

@easymac
Copy link
Contributor

easymac commented Feb 18, 2016

I'm on the side of adding visible hostmasks that can be hidden/shown with CSS as it's an easy way to identify when someone joins under an alias or when a user is being impersonated.

@vectr0n
Copy link

vectr0n commented Feb 22, 2016

+1 Would love to see at least the option to show the ident@host in join/part.

@Jay2k1
Copy link
Member

Jay2k1 commented Feb 22, 2016

A huge +1. This is a must-have for me.

@richrd
Copy link
Member

richrd commented Feb 23, 2016

Big 👍 for the possibility for themes to show the hostmask.
Could be off by default but would be nice if it would be an option.

@astorije
Copy link
Member

Added in #94, closing.

@astorije astorije added this to the 1.3.0 milestone Apr 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature Tickets that describe a desired feature or PRs that add them to the project.
Projects
None yet
Development

No branches or pull requests

8 participants