I just looked at 11_buddy_list_really_show.patch--yikes.  It does things
that are very heavy-handed.  The application really should not do all
this stuff just to show the buddy list:

gtk_widget_show(GTK_WIDGET(gtkblist->window));
gtk_window_deiconify(GTK_WINDOW(gtkblist->window));
gtk_window_present(GTK_WINDOW(gtkblist->window));
gtk_window_set_keep_above(GTK_WINDOW(gtkblist->window), TRUE);
gtk_window_stick(GTK_WINDOW(gtkblist->window));

It's... kind of ridiculous.  Without this patch Pidgin calls only
gtk_window_present.  The documentation for this function is quite clear
on its behavior:
http://library.gnome.org/devel/gtk/stable/GtkWindow.html#gtk-window-
present

I'm curious to know who wrote this patch, and if that person has prior
experience working on gtk applications.  I am not familiar with the
problem that this patch attempts to fix, but I'm quite sure that this
patch is not the correct solution.  An application should not attempt to
override the desires of the window manager.

-- 
Buddy List taskbar icon shows on all virtual desktops
https://bugs.launchpad.net/bugs/346840
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to