[Pidgin] #5876: ability to hide away/idle buddies as well as mobile buddies

Pidgin trac at pidgin.im
Sat May 31 07:10:10 EDT 2008


#5876: ability to hide away/idle buddies as well as mobile buddies
---------------------------+------------------------------------------------
  Reporter:  sumitk        |       Owner:                             
      Type:  patch         |      Status:  new                        
  Priority:  minor         |   Milestone:  Patches Needing Improvement
 Component:  pidgin (gtk)  |     Version:  2.4.2                      
Resolution:                |    Keywords:                             
   Pending:  0             |  
---------------------------+------------------------------------------------
Comment (by sadrul):

 I think something like a !FinchBlistManager
 (http://developer.pidgin.im/doxygen/dev/html/structFinchBlistManager.html)
 would be useful for Pidgin too (except 'find_parent', perhaps). That would
 allow plugins to determine whether to show a particular blist-node or not
 with finer granularity (with the current behaviour as the default, of
 course). So there could be a plugin that could hide idle buddies, away
 buddies (for its own definition of 'away'ness), idle or away buddies,
 mobile buddies, buddies who are not listening to any songs etc. etc.

 Thoughts?

 About the API additions: some of them (e.g.
 purple_blist_get_non_away_online_count) look very special purpose, and
 doesn't need to be in the API at all. (also, you may want to look at
 purple_blist_node_next).

 purple_blist_node_get_bool_with_default looks like a good idea (the
 documentation for it is not quite correct, though ;) ). We should probably
 have those for _get_int and _get_string too. Perhaps there should also be
 a 'purple_blist_node_has_setting' to check whether a particular setting is
 set for a blistnode. Or it could return a PurpleValue for the setting
 (NULL when none is set). That would help in pref-migration if the type of
 a setting is changed.

 purple_presence_away doesn't look all that good (especially with the weird
 include-parameters). I agree with rlaager that checking for online and
 !available should do.

-- 
Ticket URL: <http://developer.pidgin.im/ticket/5876#comment:5>
Pidgin <http://pidgin.im>
Pidgin


More information about the Tracker mailing list