[PATCH 3/4] 3 Save iconpath if icon will be used

2012-11-28 Thread Rodolfo García Peñas
>From 244671a650bae1be4d2b12d0ecc308c63b08783d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?"Rodolfo=20Garc=C3=ADa=20Pe=C3=B1as=20(kix)"?= Date: Wed, 28 Nov 2012 23:56:19 +0100 Subject: [PATCH 3/4] 3 Save iconpath if icon will be used This patch avoid to save the icon name in the confi

Re: [PATCH 3/4] 3 Save iconpath if icon will be used

2012-11-29 Thread BALATON Zoltan
On Thu, 29 Nov 2012, Rodolfo Garc?a Pe?as wrote: This patch avoid to save the icon name in the configuration file if the flag "Ignore client suplied icon" is not selected. Sorry but if the icon set by the user is not used when Ignore flag is not set then why do we have this option at all? +

Re: [PATCH 3/4] 3 Save iconpath if icon will be used

2012-11-29 Thread kix
On 2012-11-29 11:03, BALATON Zoltan wrote: On Thu, 29 Nov 2012, Rodolfo Garc?a Pe?as wrote: This patch avoid to save the icon name in the configuration file if the flag "Ignore client suplied icon" is not selected. Sorry but if the icon set by the user is not used when Ignore flag is not set t

Re: [PATCH 3/4] 3 Save iconpath if icon will be used

2012-11-29 Thread BALATON Zoltan
On Thu, 29 Nov 2012, Rodolfo Garc?a Pe?as (kix) wrote: Yes, you are right. We have four options: 1. ignore flag + file set 2. no ignore flag + file set 3. ignore flag + no file set 4. no ignore flag + no file set Option 1 is clear, we provide a new icon and ignore the supplied icon by the clie

Re: [PATCH 3/4] 3 Save iconpath if icon will be used

2013-01-24 Thread Rodolfo García Peñas
On Thu, 29 Nov 2012, BALATON Zoltan escribió: > On Thu, 29 Nov 2012, Rodolfo Garc?a Pe?as (kix) wrote: > >Yes, you are right. We have four options: > > > >1. ignore flag + file set > >2. no ignore flag + file set > >3. ignore flag + no file set > >4. no ignore flag + no file set > > > >Option 1 is

Re: [PATCH 3/4] 3 Save iconpath if icon will be used

2013-01-25 Thread Carlos R. Mafra
On Fri, 25 Jan 2013 at 1:06:58 +0100, Rodolfo García Peñas wrote: > I changed the behavior for the winspector. Can you test it now? > > More testers are welcome. If the patch is ok, I will write the commit info. I'm a bit lost about which behavior is being fixed. What is the test case to aim f

Re: [PATCH 3/4] 3 Save iconpath if icon will be used

2013-01-25 Thread BALATON Zoltan
On Fri, 25 Jan 2013, Carlos R. Mafra wrote: On Fri, 25 Jan 2013 at 1:06:58 +0100, Rodolfo García Peñas wrote: I changed the behavior for the winspector. Can you test it now? More testers are welcome. If the patch is ok, I will write the commit info. I'm a bit lost about which behavior is bei

Re: [PATCH 3/4] 3 Save iconpath if icon will be used

2013-01-26 Thread Rodolfo García Peñas (kix)
On 25/01/2013 11:53, BALATON Zoltan wrote: > On Fri, 25 Jan 2013, Carlos R. Mafra wrote: >> On Fri, 25 Jan 2013 at 1:06:58 +0100, Rodolfo García Peñas wrote: >>> I changed the behavior for the winspector. Can you test it now? >>> >>> More testers are welcome. If the patch is ok, I will write the c

Re: [PATCH 3/4] 3 Save iconpath if icon will be used

2013-01-26 Thread Carlos R. Mafra
On Sat, 26 Jan 2013 at 11:12:01 +0100, "Rodolfo García Peñas (kix)" wrote: Thanks for the detailed explanation. > On 25/01/2013 11:53, BALATON Zoltan wrote: > > > > The procedure to test Ignore app icon is here: > > http://lists.windowmaker.org/dev/msg04452.html > I am changing the icon behavi

Re: [PATCH 3/4] 3 Save iconpath if icon will be used

2013-01-26 Thread Rodolfo García Peñas
On Fri, 25 Jan 2013, BALATON Zoltan escribió: > On Fri, 25 Jan 2013, Carlos R. Mafra wrote: > >On Fri, 25 Jan 2013 at 1:06:58 +0100, Rodolfo García Peñas wrote: > >>I changed the behavior for the winspector. Can you test it now? > >> > >>More testers are welcome. If the patch is ok, I will write

Re: [PATCH 3/4] 3 Save iconpath if icon will be used

2013-01-26 Thread Rodolfo García Peñas
On Sat, 26 Jan 2013, Rodolfo kix Garcia escribió: > On Fri, 25 Jan 2013, BALATON Zoltan escribió: > > > On Fri, 25 Jan 2013, Carlos R. Mafra wrote: > > >On Fri, 25 Jan 2013 at 1:06:58 +0100, Rodolfo García Peñas wrote: > > >>I changed the behavior for the winspector. Can you test it now? > > >>

Re: [PATCH 3/4] 3 Save iconpath if icon will be used

2013-01-27 Thread Rodolfo García Peñas
On Fri, 25 Jan 2013, BALATON Zoltan escribió: > On Fri, 25 Jan 2013, Carlos R. Mafra wrote: > >On Fri, 25 Jan 2013 at 1:06:58 +0100, Rodolfo García Peñas wrote: > >>I changed the behavior for the winspector. Can you test it now? > >> > >>More testers are welcome. If the patch is ok, I will write

Re: [PATCH 3/4] 3 Save iconpath if icon will be used

2013-01-27 Thread Rodolfo García Peñas
On Sun, 27 Jan 2013, Rodolfo kix Garcia escribió: > On Fri, 25 Jan 2013, BALATON Zoltan escribió: > > > On Fri, 25 Jan 2013, Carlos R. Mafra wrote: If you want see the code. kix -- ||// //\\// Rodolfo "kix" Garcia ||\\// //\\ http://www.kix.es/ ptchs.tgz Description: application/gtar-compres

Re: [PATCH 3/4] 3 Save iconpath if icon will be used

2013-01-28 Thread BALATON Zoltan
Hello, On Sun, 27 Jan 2013, Rodolfo García Peñas wrote: better, but... I would like to know your oppinion before send the patches. My opinion is as before: code cleanup patches should not change behaviour so everything should work the same after your patches as they worked before them. (The