Youness Alaoui wrote: > Hi, > Thanks for the patch and for giving a detailed explanation > of your changes... > however, I see two problems : > 1 - code duplication.. it would be better to have the code > that checks for the system configs in config.tcl after we > loadProfile... this way, the code will be there only one... > Also, make sure that the option for 'use system settings' > would only be available when it is actually possible to > check user settings (on mac, windows, do not show the > option).. > also note that it might be confusing if someone has old > settings for gnome, and is using KDE, and amsn takes gnome > settings instead of his new KDE settings... maybe check for > the $::env(DESKTOP) variable too... > > Thanks again for the contribution! > > KaKaRoTo >
I had not thought in that cases but it's okay! I'll inform you when it's ready :) The first problem is that $DESKTOP is not defined (at least for me in GNOME). So I don't know how to guess the windows manager that is running in that moment. Any other idea? -- Pablo. ------------------------------------------------------------------------------ _______________________________________________ Amsn-devel mailing list Amsn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/amsn-devel
