Bug 335796

Summary:	yast2 gtk should use gtk_window_set_transient_for() for dialogs
Product:	[openSUSE] openSUSE 10.3	Reporter:	Stefan Sauer <ensonic>
Component:	YaST2	Assignee:	Forgotten User h13THG8RK1 <forgotten_h13THG8RK1>
Status:	RESOLVED INVALID	QA Contact:	Jiri Srain <jsrain>
Severity:	Minor
Priority:	P5 - None	CC:	forgotten_h13THG8RK1
Version:	Final
Target Milestone:	---
Hardware:	Other
OS:	Other
Whiteboard:
Found By:	---	Services Priority:
Business Priority:		Blocker:	---
Marketing QA Status:	---	IT Deployment:	---
Attachments:	test case YCP test case

Description Stefan Sauer 2007-10-22 19:53:01 UTC

dialogs in yast2 gtk programms jump around on the screen. If the would have been marked as transient, window-managers would center/place them over their parent window.

Comment 1 Forgotten User h13THG8RK1 2007-10-29 17:31:52 UTC

Created attachment 181071 [details]
test case

Hi, Stefan. Please test this program. It creates a window, then after 4 secs a transient window to that. Tell me if it is different than what you are getting with yast.

Also, just to be sure, you are running Gnome, right? And whats the window manager? Metacity or Compiz (gets activated with Xgl)?

Comment 2 Stefan Sauer 2007-10-29 19:13:11 UTC

yes that it. This way the window open above the parent and not at random locations.

a second issue is that eveytime you open a dialog, you steal the focus. See
http://osdir.com/ml/gnome.gaim.devel/2004-12/msg00077.html

It suggests to do 
popup_window.set_focus_on_map (False)
as well. Unfortunately there is a bug in emerald (compiz) that dows not show the new window above (files as Bug #337652).

Comment 3 Forgotten User h13THG8RK1 2007-10-29 19:52:21 UTC

Created attachment 181093 [details]
YCP test case

Wasn't expecting that... To be clear, tell me in what tool this happens and what dialog (I'm guessing the several popup dialogs from the software manager? :)).

And your complaint is that the windows that popup aren't being centered to the main window. You are NOT complaining that the popup windows steal focus.

If the window's title is empty, it should definitively be transient, as those two properties are set together.

This is the YCP code that should behave just like the previous GTK+ test case. To run it: /usr/lib/YaST2/bin/y2base TransientDialog.ycp gtk

Doesn't it behave the same for you? (only only that the window sizes are different) Try also to pass "qt"...

Comment 4 Stefan Sauer 2007-10-29 21:37:53 UTC

Thanks! It works fine. Regarding the focus, there is Bug #326158. I'll discuss that matter there. Thanks for working on fixing the placement though.

Comment 5 Forgotten User h13THG8RK1 2007-10-29 23:29:28 UTC

Thanks for testing the YCP program. It means that not everything is broken. ;)
Now, if you can reproduce the bug at will, you need to tell me where this happens. There isn't just one vector of code that could be responsible for it...

Comment 6 Stefan Sauer 2007-10-30 20:01:51 UTC

Hmm. I cannot reproduce it anymore in yast :(. I got new yast-gtk and new compiz in the meantime. I close the report as invalid then.