On Wed, May 07, 2008 at 03:57:59PM +0200, Enrico Tröger wrote:
> On Tue, 6 May 2008 11:13:33 -0300, Damián Viano <[EMAIL PROTECTED]> wrote:
>
> Hi,
>
> > Btw, I've been looking into #428020 and have some stuff but
> > I'm not sure if the problem is reproducible with other windows
> > managers (i.e. the current gnome one, whatever that is, or xfce or
> > other than fluxbox fwiw)... Have you guys reproduced it? (feel free
> > to move this to the list if you want)
>
> It doesn't happen with the SVN version of xfwm4 (I'm using it and
> tested at the time when this bug was reported and tested now again). I
> also installed a fresh Lenny with Gnome 2.22.1 and I wasn't able to
> reproduce it there too.
This makes me look into fluxbox, specifically since I can reproduce
this with fluxbox.
> > I attach a patch that fix most of this problems, except for
> > the fonts dialog launched from the font buttons in the preference
> > dialog, which I don't know how can be fixed without rewriting the
> > handler for the click in that buttons.
> One solution would be to use a normal GtkButton and create the font
> dialog manually with a own click handler (i.e. re-use the code for the
> font dialog in dialogs.c).
>
> > I'm still not too sure if adding the
> > gdk_x11_window_move_to_current_desktop() call manually is the best
> > way to handle it, but since I was trying if it worked I have the
> > patch around :-)
>
> So you can reproduce it?
Yes, I can. With fluxbox from sid (1.0.0+deb1-6), and is an already
reported bug, and fixed in upstream git repository.
> Attached is a modified version of your patch which adds
> ui_window_present() as a wrapper for gtk_window_present(). And it calls
> gdk_x11_window_move_to_current_desktop() only when compiled on systems
> with X to not break Windows/MacOSX support.
This approach was cleaner indeed, anyway is not necessary now. However
the changes of gtk_widget_show for gtk_window_present might be worth to
include to let the WM now that this dialogs should get attention, maybe?
I think it makes sense.
> Not sure whether I really got the point but it seems more like a window
> manager issue than a bug in Geany, IMO. So, I'm not very keen on fixing
> window manager problems in Geany...
Absolutely agree, I'll reassign it to fluxbox, mark as duplicate
since it is already reported, and tag it as fixed upstream.
My only question is: was Brandon using fluxbox also or is this same
bug is present in other window managers? Brandon, could you comment?
--
Damián Viano(Des) ¯ ¯ - _ _ - ¯ ¯
GPG: 0x6EB95A6F Debian ¯-_GNU_-¯ Linux
Web: http://damianv.com.ar/ ¯-¯
--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]