Skip to content
This repository has been archived by the owner on Nov 21, 2024. It is now read-only.

One-on-one profile pane #272

Closed
wants to merge 13 commits into from
Closed

One-on-one profile pane #272

wants to merge 13 commits into from

Conversation

benlangfeld
Copy link
Member

Replaces the roster pane in a one-on-one chat window with a profile pane similar to HipChat.

Builds on #268. That should be merged first.

This may require some styling work.

@benlangfeld
Copy link
Member Author

@mweibel In a lot of this work I'm coming up against a requirement for certain features to be present in Candy core (basic presence routing, etc) which rely on concepts of a general chat client (with a main roster, individual contacts, etc) that Candy doesn't have (some of which we have implemented in plugins).

Are you actively against some of these things being brought in to Candy core to make it a more general and capable XMPP client (noting that I do not necessarily mean any UI changes, but purely laying the state foundations for easily building these as plugins)? If your general stance is "do it in plugins", what do I do when there's such a circular dependency between plugin -> candy core -> plugin?

@mweibel
Copy link
Member

mweibel commented Jul 24, 2014

About your question: I basicly misunderstood what you wanted with this other PR which I said to do as a plugin. I think it's appropriate to bring in more XMPP features even if they are not directly MUC-related. UI changes we can always discuss, but they might be better in plugins.

@benlangfeld
Copy link
Member Author

Excellent, expect a deluge of PRs adding more core XMPP functionality. Thanks for your input :)

@mweibel
Copy link
Member

mweibel commented Jul 24, 2014

Neat :)
We'll talk about contributors access to candy core after some more PRs ;)

@mweibel
Copy link
Member

mweibel commented Jul 24, 2014

Btw. Off Topic: Do you think it would help if we'd use Gitter (https://gitter.im/candy-chat) for better communication in certain cases?

@mweibel
Copy link
Member

mweibel commented Jul 29, 2014

Is this on hold as well right now?

@benlangfeld
Copy link
Member Author

Yes, I'll be submitting a separate PR for some additions of roster stuff to Candy core to make this easier to implement properly.

@benlangfeld benlangfeld mentioned this pull request Aug 4, 2014
5 tasks
@benlangfeld
Copy link
Member Author

@mweibel Would you be amenable to using https://github.com/nilclass/vcardjs to replace the buggy vcard representation here?

@mweibel
Copy link
Member

mweibel commented Aug 6, 2014

Yeah might be better I guess

@mweibel
Copy link
Member

mweibel commented Jan 14, 2015

What's the status here?

@mweibel
Copy link
Member

mweibel commented Feb 26, 2015

Closing as no update has been happening. Reopen if you don't agree.

@mweibel mweibel closed this Feb 26, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants