[Gaim-patches] [ gaim-Patches-1163319 ] UI for gaim account option lists

[SourceForge.net <no...@so...>]

Patches item #1163319, was opened at 2005-03-14 16:27
Message generated for change (Comment added) made by rizzo
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1163319&group_id=235

Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Don Seiler (rizzo)
Assigned to: Sean Egan (seanegan)
Summary: UI for gaim account option lists

Initial Comment:
gaim account option lists currently have no UI.  I've
implemented one here.  I've also changed the account
option list API to take a GList of GaimKeyValuePair
structs (also included in this patch) rather than a
GList of key-then-value-then-key-then-value-etc. strings.

Patch includes the implementation of the UI in
add_protocol_options(), and the call back function in
ok_account_prefs_cb().  The destroyer that had already
been implemented should still apply in this case given
static strings.

This makes use of the GtkComboBox, which I'm told will
be available in HEAD, and this was diffed from HEAD.

Here is an example use from the PRPL init_plugin():

GaimKeyValuePair *kvp;
GList *server_list = NULL;

kvp = g_new(GaimKeyValuePair, 1);
kvp->key = _("Asia");
kvp->value = "asia.battle.net";
server_list = g_list_append (server_list, kvp);

kvp = g_new(GaimKeyValuePair, 1);
kvp->key = _("Europe");
kvp->value = "europe.battle.net";
server_list = g_list_append (server_list, kvp);

kvp = g_new(GaimKeyValuePair, 1);
kvp->key = _("US - East");
kvp->value = "useast.battle.net";
server_list = g_list_append (server_list, kvp);

option = gaim_account_option_list_new(_("Server List"),
"server_list", server_list);
prpl_info.protocol_options =
g_list_append(prpl_info.protocol_options, option);


----------------------------------------------------------------------

>Comment By: Don Seiler (rizzo)
Date: 2005-03-14 16:41

Message:
Logged In: YES 
user_id=61900

Just a follow-up note that this is really part one of a
two-part change.  Part two involves using an array instead
of a GList to pass the kvps, and then making the PRPL
responsible for destruction of that array and any other
option-related variables, which would be done in the plugin
unload() function.

See the recent email thread started by me and added-to by Ethan.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1163319&group_id=235