[Pidgin] #5521: gtk_window_present unreliable for showing buddy list on tray icon click

Pidgin trac at pidgin.im
Tue Apr 15 13:45:40 EDT 2008


#5521: gtk_window_present unreliable for showing buddy list on tray icon click
---------------------------+------------------------------------------------
  Reporter:  gagern        |       Owner:  deryni                    
      Type:  defect        |      Status:  new                       
  Priority:  minor         |   Milestone:                            
 Component:  pidgin (gtk)  |     Version:  2.4.1                     
Resolution:                |    Keywords:  tray icon, kde, gtk, focus
   Pending:  1             |  
---------------------------+------------------------------------------------
Changes (by deryni):

  * owner:  => deryni
  * pending:  0 => 1

Comment:

 pidgin is not generally in the business of hacking around broken window
 manager policies, which is exactly what this patch is doing. The
 documentation for gtk_window_present is exactly what pidgin wants to have
 happen. Specifically it "[p]resents a window to the user" which "may mean
 raising the window in the stacking order, deiconifying it, moving it to
 the current desktop, and/or giving it the keyboard focus, possibly
 dependent on the user's platform, window manager, and preferences". The
 issue here is that metacity has a policy you don't like, a policy that
 they admit doesn't work in all cases (possibly only not in the case of
 pidgin and other IM clients). The point here is that pidgin is doing the
 right thing and hacking around the metacity policy decision for
 gtk_window_present will explicitely disallow any other window manager from
 doing anything more intelligent on a tray icon click. It is exactly this
 policy-in-the-hands-of-the-application that the ewmh is supposed to help
 alleviate/prevent.

 The only bug here is that metacity is a minimalistic window manager and
 that is therefore doesn't allow per-application/per-window policy
 decisions.

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


More information about the Tracker mailing list