[Pidgin] #3319: MSN group name display support
Pidgin
trac at pidgin.im
Tue May 26 16:42:29 EDT 2009
#3319: MSN group name display support
------------------------------------+---------------------------------------
Reporter: manphiz | Owner: khc
Type: patch | Status: new
Milestone: Patches Needing Review | Component: MSN
Version: 2.2.0 | Resolution:
Keywords: msn group |
------------------------------------+---------------------------------------
Comment(by deryni):
The msn-pecan patch is not "good" but it does "work". The choices as I see
them currently are:
1. Do nothing, leaves things as broken as they are now rendering these
group chats unusable.
1. 'Tag' each message with the P4-Context header as a prefix (so
<timestamp> GroupName: P4Name: message) or some other similar tagging
method.
1. Use a patch like the one Felipe has in msn-pecan which sets the user
settable alias for the buddy to the value of the P4-Context header for the
duration of the message and then sets it back to whatever value it had
previously afterwards.
1. Determine what would be involved to allow libpurple to support
additional tags of this sort (and potentially other sorts) for incoming
(and outgoing?) messages.
These options are listed in roughly the order they are easy to implement
(I have placed the tag option about the alias patch option despite a patch
for the latter existing because I think the tagging is less of an ugly
hack).
I think the first is a non-option at this point, I think the last is
potentially useful but requires a longer timespan than is likely helpful,
and between the middle two I like the tagging better than aliasing because
it seems like less of a hack to me (as it doesn't potentially trigger tab
title changes, window title changes, buddy list refreshing, etc.).
A decision needs to be made on this point and some code committed, I have
given my preference but am not going to make the decision as I don't use
MSN in any real way.
--
Ticket URL: <http://developer.pidgin.im/ticket/3319#comment:56>
Pidgin <http://pidgin.im>
Pidgin
More information about the Tracker
mailing list