Support personal messages

Bug #1203845 reported by Mario Guerriero
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Cable
Fix Released
High
Julien Spautz

Bug Description

There should be the support for personal messages.

Revision history for this message
Julien Spautz (julien-spautz) wrote :

How should this be implemented GUI-wise?

I guess double clicking a username should add it to the channel list on the left and open a new chat.
Closing could be done by using the remove button in the inline toolbar.

Any other suggestions?

Changed in cable:
status: New → Confirmed
importance: Undecided → Critical
importance: Critical → High
Revision history for this message
Mario Guerriero (mefrio-g) wrote :

I think that double clicking or receiving a personal message should add a new "Channel list" called "Messages" and below that a list of the users who are interacting with us.

Revision history for this message
982c80311320c1b (alexander-wilms) wrote :

Additionally, a context menu entry in the user list could be added

Changed in cable:
milestone: none → 0.1
Eduard Gotwig (gotwig)
Changed in cable:
assignee: nobody → Eduard Gotwig (gotwig)
Revision history for this message
Eduard Gotwig (gotwig) wrote :

Please see https://blueprints.launchpad.net/cable/+spec/channels-pms for a blueprint.

For something like thi,s we have to rethink how to represent a list entry in Cable.

Revision history for this message
Julien Spautz (julien-spautz) wrote :

Just some help on how to implement this properly.

We should share as much code between normal channels and private chats. We need to create a base class for them (e.g. Chat, Conversation...) and derive Channel and PrivateChat from that base class. All shared code goes into the base class (obviously).

This has to be done in /src/Controller, /src/View, and /src/Widgets.

I'll have a look at this when I'm back to make sure we don't have unnecessary redundancy or bad OOP design in the code.

Changed in cable:
assignee: Eduard Gotwig (gotwig) → Julien Spautz (julien-spautz)
Revision history for this message
Julien Spautz (julien-spautz) wrote :

See the related blueprint for a more detailed spec. Comments are welcome.

Changed in cable:
status: Confirmed → In Progress
Changed in cable:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Related blueprints

Remote bug watches

Bug watches keep track of this bug in other bug trackers.