Re: PR for Bug 55258

[Philippe Mouawad]

Thanks for feedback.
Integrated.
To be uniform, I added new to menu.


On Sun, Jan 22, 2017 at 2:27 PM, Felix Schumacher <
felix.schumac...@internetallee.de> wrote:

> Am Sonntag, den 22.01.2017, 14:00 +0100 schrieb Philippe Mouawad:
> > Hello,
> > Any thoughts on this one ?
>
> While I wouldn't mind keeping both icons, it looks like the majority
> wanted to have only one icon and that one should be the "new" icon.
>
> In that light, it looks like the PR is OK.
>
> (I haven't looked at the docs, whether it is mentioned there, though)
>
> Felix
>
> > Thanks
> >
> > On Sun, Jan 15, 2017 at 2:55 PM, Philippe Mouawad <
> > p.moua...@ubik-ingenierie.com> wrote:
> >
> > >
> > > Hello,
> > > Shall we merge :
> > > https://github.com/apache/jmeter/pull/227
> > >
> > > Regards
> > > Philippe
> > >
> > >
> >
>



-- 
Cordialement.
Philippe Mouawad.

Re: PR for Bug 55258

[Felix Schumacher]

Am Sonntag, den 22.01.2017, 14:00 +0100 schrieb Philippe Mouawad:
> Hello,
> Any thoughts on this one ?

While I wouldn't mind keeping both icons, it looks like the majority
wanted to have only one icon and that one should be the "new" icon.

In that light, it looks like the PR is OK.

(I haven't looked at the docs, whether it is mentioned there, though)

Felix

> Thanks
> 
> On Sun, Jan 15, 2017 at 2:55 PM, Philippe Mouawad <
> p.moua...@ubik-ingenierie.com> wrote:
> 
> > 
> > Hello,
> > Shall we merge :
> > https://github.com/apache/jmeter/pull/227
> > 
> > Regards
> > Philippe
> > 
> > 
> 

Re: PR for Bug 55258

[Philippe Mouawad]

Hello,
Any thoughts on this one ?
Thanks

On Sun, Jan 15, 2017 at 2:55 PM, Philippe Mouawad <
p.moua...@ubik-ingenierie.com> wrote:

> Hello,
> Shall we merge :
> https://github.com/apache/jmeter/pull/227
>
> Regards
> Philippe
>
>


-- 
Cordialement.
Philippe Mouawad.
Ubik-Ingénierie

UBIK LOAD PACK Web Site 

UBIK LOAD PACK on TWITTER 

PR for Bug 55258

[Philippe Mouawad]

Hello,
Shall we merge :
https://github.com/apache/jmeter/pull/227

Regards
Philippe