In some rare cases, it can happen that the minimal dialog is open twice. For instance, if a user opens it and then connects a new display, it is possible that he ends up with two dialogs, which is undesirable. Instead, the xfsettingsd should check if the minimal dialog is already running and either just not open another one or give the existing instance focus.
This actually happens quite often here, and all you need to do is remove/reinsert the HDMI connector, or sometimes simply touch the cable if the connector is fragile...
Created attachment 6986 patch Does this patch accomplish the desired effect?
I would like to test this patch, but I'm unable to ./configure master: checking for exo-2 >= 0.11.0... not found *** The required package exo-2 was not found on your system. *** Please install exo-2 (atleast version 0.11.0) or adjust *** the PKG_CONFIG_PATH environment variable if you *** installed the package in a nonstandard prefix so that *** pkg-config is able to find it. Is it still possible to build against Gtk2, or does it work only with Gtk3 now?
xfce4-settings has been ported to Gtk3 only since September and this patch makes use of GtkApplication which is not available in Gtk2.
Hi, You can do the same with glib Gapplication. (Gtk already use it) but it is newer than the dependencies of xfce4-settings, so it should be optional in compilation time.. https://developer.gnome.org/gio/stable/GApplication.html
Matias: I'm already using GApplication's functions, replacing GtkApplication/gtk_application_new by GApplication/g_application_new is possible, the current dependencies are compatible.
> Matias: I'm already using GApplication's functions, replacing GtkApplication/gtk_application_new by GApplication/g_application_new is possible, Of course.. This is what I meant.. ;) > the current dependencies are compatible. In master yes, but if you want to apply it to the stable version like me, no.. =) xfce4-settings stable deppend on glib 2.24.0, and GApplication was added on 2.28. Of course this in 99% of the time is not a problem.. Just a note .. In case you want to release a minor update of the stable .. ;) Regards, Matias
(In reply to André Miranda from comment #4) > xfce4-settings has been ported to Gtk3 only since September and this patch > makes use of GtkApplication which is not available in Gtk2. Hmm, Xfce seems to have gone uncharacteristically aggressive with the Gtk3 requirements. I've tried to build xfce4-settings and it fails like this: *** The required package gtk+-3.0 was found on your system, *** but the installed version (3.18.9) is too old. *** Please upgrade gtk+-3.0 to atleast version 3.20.0, or adjust *** the PKG_CONFIG_PATH environment variable if you installed *** the new version of the package in a nonstandard prefix so *** pkg-config is able to find it. Not even Ubuntu Xenial ships 3.20... And that isn't exactly ancient, as it's the current LTS for Ubuntu.
@Liv, maybe we're all using Arch? :P Try to set the GTK version to 3.18 in configure.ac.in, perhaps nothing breaks and for the purpose of testing the worse thing that can happen is a compilation error. If everything works fine you can try to convince the devs to downgrade the requirements.
(In reply to André Miranda from comment #9) > @Liv, maybe we're all using Arch? :P > Try to set the GTK version to 3.18 in configure.ac.in, perhaps nothing > breaks and for the purpose of testing the worse thing that can happen is a > compilation error. If everything works fine you can try to convince the devs > to downgrade the requirements. Thanks. For now I've simply managed to find Simon's Gtk3 package for Xenial, and am building packages in my PPA against that: https://launchpad.net/~landronimirc/+archive/ubuntu/xfce-gtk3 I'm now slowly collecting all packages required for Gtk3 Xfce. We'll see how that works out.
@Sean: mind to review? Looks simple enough.
I can confirm that the patch works.
Andre Miranda referenced this bugreport in commit 786c0dc49dd10b092d1bf6d91223d8e148800146 Make minimal dialog single instance (Bug #11169) https://git.xfce.org/xfce/xfce4-settings/commit?id=786c0dc49dd10b092d1bf6d91223d8e148800146
Thanks for the patch! Fix applied in the above commit.