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
Add favoriting/pinning channels #4556
base: master
Are you sure you want to change the base?
Conversation
Sounds like the channel object should know which network it belongs to, so it should just work™. |
@xPaw I meant the command interface:
The alternative is |
Ah right, misunderstood. The (un)favorite syntax would match all the existing commands. |
it's always possible to have those be aliases i for instance add aliases in git for |
I'd give a vote for |
Let's go for consistency and /favorite + /unfavorite then? Or what's the consensus now? |
I agree about /favorite and /unfavorite. 👍 Good work @MaxLeiter! |
Closes #180, #4078(?)
It effectively adds a new network, 'favorites' that can contain channels from any network.
If there's a name conflict, the server name is appended to the channel name while it's favorited.
The functionality all works except the command, as I'm not sure of how that should work.
I was thinking:
/favorites add [target] [...targets]
/favorites remove [target] [...targets]
/favorites clear
If no target is provided, it's assumed to be the current channel.
Here you can see two
#testing123
channels with the server name attached. It's also added to the topic line, so (for example) on mobile you can be confident which you pressedThe Favorites 'network' can't be opened (clicking it does nothing), but does have a context menu:
You can add channels/queries to your favorites with the context menu (or aforementioned command):