https://bugzilla.novell.com/show_bug.cgi?id=436718 JP Rosevear <jpr@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|bnc-team-gnome@forge.provo.novell.com |sreeves@novell.com Priority|P5 - None |P2 - High -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=436718 User tambet@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=436718#c2 Tambet Ingo <tambet@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tambet@novell.com Status|NEEDINFO |NEW Info Provider|tambet@novell.com | --- Comment #2 from Tambet Ingo <tambet@novell.com> 2008-11-03 01:49:49 MDT --- No, this is the first time I hear about this bug. Reading the main-menu code, I found out it uses nm-applet.glade file for this, which had incompatible changes in May. I don't think it's a good idea to use other application's glade files, there's no guarantees it won't change (yeah-yeah, like the public API hasn't change :) but at least it will be declared stable at some point, which is not the case for the glade file). I think the network status functionality in main-menu should be reconsidered in general. Is it really useful to see the same status in two locations (always visible status icon and opened main menu)? Especially when the main-menu one knows how to handle a single active connection and doesn't know anything about VPN (ie, has less information). If we think we do want it, it might be better to reorganize the code in the applet to create a stand alone executable for the information dialog. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=436718 User sreeves@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=436718#c4 --- Comment #4 from Scott Reeves <sreeves@novell.com> 2008-11-20 09:35:38 MST --- I agree that the current implementation of directly using NM's glade file will always be a maintenance problem. We also currently have different behavior of that tile when running with or without NM. I propose that we change this to launch the various config utilities when clicked. Running without NM - launch lan.desktop Running with NM - launch nm-connection-editor.deskotp I have a patch that does this. Okay to commit ? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=436718 User tambet@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=436718#c6 --- Comment #6 from Tambet Ingo <tambet@novell.com> 2008-11-21 00:09:20 MST --- nm-connection-editor doesn't show you the active connections, it's like launching yast for non-NM case. (So I don't think it's a good idea). nm-applet doesn't register a DBus owner, so there's no existing place to add methods. I would just copy+paste the existing code from nm-applet to main menu. There's no need for glade file at all as it only contains a GtkDialog and a GtkNotebook inside it. There won't be any issues with API stability, at least for code 11 as NM is API stable now (there's been two RC releases and the final 0.7 scheduled for this weekend). -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=436718 User tambet@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=436718#c8 --- Comment #8 from Tambet Ingo <tambet@novell.com> 2008-11-21 05:27:20 MST --- Ok, so again, why do we have the network information there at all? How is it more convenient to anyone to open a dialog to see the same information that is already present on the tray without clicking anywhere? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=436718 User sreeves@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=436718#c10 --- Comment #10 from Scott Reeves <sreeves@novell.com> 2008-11-25 12:48:37 MST --- I think we need to be consistent when clicking the tile. Currently in 11.0 when running NM we show a status type box, when running non NM we show an editor type box. I agree that a status type box is better but to do that we need to cut/paste NM code which will always be a maintainance issue, and we don't have any such capability right now for the non NM case. The patch in comment #4 provides consistent behavior and does not have these issues. For 11.1 I submitted this patch. We should probably discuss this more for SLE -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=436718 User sreeves@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=436718#c12 Scott Reeves <sreeves@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED --- Comment #12 from Scott Reeves <sreeves@novell.com> 2009-01-14 16:14:29 MST --- Fixed in 11.1 release. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.