Mercurial > pidgin
changeset 17451:0e4a789cd027
We use a timeout in our blist selection changed callback to prevent buddy
list flickering when scrolling through the list, but this appears to be the
root cause of buddies sometimes being rendered "half height". Ticket #1511
managed to narrow it down to happening when collapsing (or expanding) a
group below the currently selected node, so I made it so the timeout is
only used when the new selection is not a group node. Fixes #1511
| author | Stu Tomlinson <stu@nosnilmot.com> |
|---|---|
| date | Sun, 03 Jun 2007 18:22:15 +0000 |
| parents | 010e8f32a0af |
| children | 407b25690442 |
| files | pidgin/gtkblist.c |
| diffstat | 1 files changed, 9 insertions(+), 2 deletions(-) [+] |
line wrap: on
line diff
--- a/pidgin/gtkblist.c Sun Jun 03 01:05:59 2007 +0000 +++ b/pidgin/gtkblist.c Sun Jun 03 18:22:15 2007 +0000 @@ -4737,8 +4737,15 @@ NODE_COLUMN, &new_selection, -1); } - /* we set this up as a timeout, otherwise the blist flickers */ - g_timeout_add(0, (GSourceFunc)do_selection_changed, new_selection); + /* we set this up as a timeout, otherwise the blist flickers ... + * but we don't do it for groups, because it causes total bizarness - + * the previously selected buddy node might rendered at half height. + */ + if ((new_selection != NULL) && PURPLE_BLIST_NODE_IS_GROUP(new_selection)) { + do_selection_changed(new_selection); + } else { + g_timeout_add(0, (GSourceFunc)do_selection_changed, new_selection); + } } static gboolean insert_node(PurpleBuddyList *list, PurpleBlistNode *node, GtkTreeIter *iter)