From kde-devel  Mon Dec 14 02:31:11 2009
From: Ravi Vagadia <ravirdv () gmail ! com>
Date: Mon, 14 Dec 2009 02:31:11 +0000
To: kde-devel
Subject: Re: [Patch] bug 198661
Message-Id: <5632b930912131819x39e53d9dh7875da90289da4b8 () mail ! gmail ! com>
X-MARC-Message: https://marc.info/?l=kde-devel&m=126075723720107
MIME-Version: 1
Content-Type: multipart/mixed; boundary="--===============0961557220=="

--===============0961557220==
Content-Type: multipart/alternative; boundary=0016e6480e085513eb047aa6e787

--0016e6480e085513eb047aa6e787
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

yes placing that option under filters is not a good idea, any suggestion ?


With Regards,
Ravi Vagadia




Mike Ditka <http://www.brainyquote.com/quotes/authors/m/mike_ditka.html>  -
"If God had wanted man to play soccer, he wouldn't have given us arms."

On Sun, Dec 13, 2009 at 3:14 PM, Sebastian K=FCgler <sebas@kde.org> wrote:

> On Saturday 12 December 2009 19:51:51 Ravi Vagadia wrote:
> > ah.. unfortunately i'm quite late for it to make into 4.4... anyway i'd
> > release it as separate applet then.
>
> Noooooo! Please don't. Releasing it as a separate applet will create lots
> of
> confusion and problems:
>
> - You need to backport fixes to the taskbar, otherwise you'll have two
> taskbars that
> looks the same, one crashy with additional features, one stable without
> - You'll confuse users and developers (which taskbar applet is being
> debugged here?)
>
> So forking central plasma components is really a bad idea, even if the
> alternative
> means waiting for trunk/ to open again. In the meantime, have your patch
> reviewed on
> the plasma-devel list, there's a plasma group on reviewboard as well.
>
> As to the patch itself. I think the new option looks a bit strange in  th=
e
> "Filters"
> section of the UI, that could surely be improved.
>
> > I'd definitely love to write more patches... and ya I'd definitely try
> for
> > svn account once I write few patches
> > thanks for encouraging words :)
>
> If you want to write more patches (and want to help with making 4.4
> awesome), we're
> now in bugfixing mode. Pick a bug, fix it, put it up on reviewboard and
> have it
> committed.
>
> As Markus said, new features will have to wait a bit, but that doesn't me=
an
> that you
> can't do something useful in the meantime. Even often bugfixing is a good
> way to get
> started with a new codebase and become part of the team.
>
> In any case, head over to the plasma-devel list. And welcome :)
> --
> sebas
>
> http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9
>
>
> >> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to
> unsubscribe <<
>
>

--0016e6480e085513eb047aa6e787
Content-Type: text/html; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

yes placing that option under filters is not a good idea, any suggestion ?<=
div><br></div><div><br clear=3D"all">With Regards,<br>Ravi Vagadia<br><br><=
br><br><br><a href=3D"http://www.brainyquote.com/quotes/authors/m/mike_ditk=
a.html" target=3D"_blank">Mike Ditka</a> =A0- &quot;If God had wanted man t=
o play soccer, he wouldn&#39;t have given us arms.&quot;
<br><br><div class=3D"gmail_quote">On Sun, Dec 13, 2009 at 3:14 PM, Sebasti=
an K=FCgler <span dir=3D"ltr">&lt;<a href=3D"mailto:sebas@kde.org">sebas@kd=
e.org</a>&gt;</span> wrote:<br><blockquote class=3D"gmail_quote" style=3D"m=
argin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div class=3D"im">On Saturday 12 December 2009 19:51:51 Ravi Vagadia wrote:=
<br>
&gt; ah.. unfortunately i&#39;m quite late for it to make into 4.4... anywa=
y i&#39;d<br>
&gt; release it as separate applet then.<br>
<br>
</div>Noooooo! Please don&#39;t. Releasing it as a separate applet will cre=
ate lots of<br>
confusion and problems:<br>
<br>
- You need to backport fixes to the taskbar, otherwise you&#39;ll have two =
taskbars that<br>
looks the same, one crashy with additional features, one stable without<br>
- You&#39;ll confuse users and developers (which taskbar applet is being de=
bugged here?)<br>
<br>
So forking central plasma components is really a bad idea, even if the alte=
rnative<br>
means waiting for trunk/ to open again. In the meantime, have your patch re=
viewed on<br>
the plasma-devel list, there&#39;s a plasma group on reviewboard as well.<b=
r>
<br>
As to the patch itself. I think the new option looks a bit strange in =A0th=
e &quot;Filters&quot;<br>
section of the UI, that could surely be improved.<br>
<div class=3D"im"><br>
&gt; I&#39;d definitely love to write more patches... and ya I&#39;d defini=
tely try for<br>
&gt; svn account once I write few patches<br>
&gt; thanks for encouraging words :)<br>
<br>
</div>If you want to write more patches (and want to help with making 4.4 a=
wesome), we&#39;re<br>
now in bugfixing mode. Pick a bug, fix it, put it up on reviewboard and hav=
e it<br>
committed.<br>
<br>
As Markus said, new features will have to wait a bit, but that doesn&#39;t =
mean that you<br>
can&#39;t do something useful in the meantime. Even often bugfixing is a go=
od way to get<br>
started with a new codebase and become part of the team.<br>
<br>
In any case, head over to the plasma-devel list. And welcome :)<br>
<font color=3D"#888888">--<br>
sebas<br>
<br>
<a href=3D"http://www.kde.org" target=3D"_blank">http://www.kde.org</a> | <=
a href=3D"http://vizZzion.org" target=3D"_blank">http://vizZzion.org</a> | =
GPG Key ID: 9119 0EF9<br>
</font><br><br>
&gt;&gt; Visit <a href=3D"http://mail.kde.org/mailman/listinfo/kde-devel#un=
sub" target=3D"_blank">http://mail.kde.org/mailman/listinfo/kde-devel#unsub=
</a> to unsubscribe &lt;&lt;<br>
<br></blockquote></div><br></div>

--0016e6480e085513eb047aa6e787--

--===============0961557220==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

 
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<

--===============0961557220==--