[Pidgin] #367: tweak x11 docklet embedding

Pidgin trac at pidgin.im
Fri Apr 27 19:18:10 EDT 2007


#367: tweak x11 docklet embedding
--------------------------+-------------------------------------------------
 Reporter:  charkins      |       Owner:     
     Type:  patch         |      Status:  new
 Priority:  minor         |   Milestone:     
Component:  pidgin (gtk)  |     Version:  2.0
 Keywords:                |  
--------------------------+-------------------------------------------------
 This patch has two changes:

 First is to slightly change the re-creation behavior of the docklet.
 Sadrul
 pointed out on gaim-devel that the x11 docklet re-creates itself when it
 is
 destroyed and thought this might be causing problems with fluxbox. This
 patch now distinguishes between initial creation and re-creation. When
 re-creating, the docklet no longer registers itself as a visibility
 manager
 until it successfully embeds.

 Second is to change the timeout behavior for embedding. Because the
 notification area API is asynchronous, gaim assumes the docklet gets
 embeded for a certain timeout period, allowing the buddy list to start
 hidden before the docklet has been embeded in the notification area. If
 the
 timeout occurs, it is removed as a visibility manager and the buddy list
 will become visible. This timeout has been set at 5 seconds. There have
 been a few reports that indicate this timeout period is not long enough
 when starting gaim from a saved session upon login. I have been hesitant
 to
 increase the timeout, as it has the potential of delaying the startup of
 gaim for the timeout period if the buddy list was hidden when gaim last
 closed and there is not currently a notification area available. This
 patch makes the x11 docklet track whether it was successfully embedded and
 uses a longer timeout (15 seconds in this patch) only if it successfully
 embedded on the previous execution. Otherwise, it uses the shorter 5
 second
 timeout.

-- 
Ticket URL: <http://developer.pidgin.im/ticket/367>
Pidgin <http://pidgin.im>
Pidgin


More information about the Tracker mailing list