[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-devel
Subject: Re: [Patch] bug 198661
From: Ravi Vagadia <ravirdv () gmail ! com>
Date: 2009-12-14 2:31:11
Message-ID: 5632b930912131819x39e53d9dh7875da90289da4b8 () mail ! gmail ! com
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
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 <<
>
>
[Attachment #5 (text/html)]
yes placing that option under filters is not a good idea, any suggestion \
?<div><br></div><div><br clear="all">With Regards,<br>Ravi \
Vagadia<br><br><br><br><br><a \
href="http://www.brainyquote.com/quotes/authors/m/mike_ditka.html" \
target="_blank">Mike Ditka</a> - "If God had wanted man to play soccer, he \
wouldn't have given us arms." <br><br><div class="gmail_quote">On Sun, Dec \
13, 2009 at 3:14 PM, Sebastian Kügler <span dir="ltr"><<a \
href="mailto:sebas@kde.org">sebas@kde.org</a>></span> wrote:<br><blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex;">
<div class="im">On Saturday 12 December 2009 19:51:51 Ravi Vagadia wrote:<br>
> ah.. unfortunately i'm quite late for it to make into 4.4... anyway \
i'd<br> > release it as separate applet then.<br>
<br>
</div>Noooooo! Please don't. Releasing it as a separate applet will create lots \
of<br> confusion and problems:<br>
<br>
- You need to backport fixes to the taskbar, otherwise you'll have two taskbars \
that<br> looks the same, one crashy with additional features, one stable without<br>
- You'll confuse users and developers (which taskbar applet is being debugged \
here?)<br> <br>
So forking central plasma components is really a bad idea, even if the \
alternative<br> means waiting for trunk/ to open again. In the meantime, have your \
patch reviewed on<br> the plasma-devel list, there's a plasma group on \
reviewboard as well.<br> <br>
As to the patch itself. I think the new option looks a bit strange in the \
"Filters"<br> section of the UI, that could surely be improved.<br>
<div class="im"><br>
> I'd definitely love to write more patches... and ya I'd definitely try \
for<br> > svn account once I write few patches<br>
> thanks for encouraging words :)<br>
<br>
</div>If you want to write more patches (and want to help with making 4.4 awesome), \
we're<br> now in bugfixing mode. Pick a bug, fix it, put it up on reviewboard and \
have it<br> committed.<br>
<br>
As Markus said, new features will have to wait a bit, but that doesn't mean that \
you<br> can't do something useful in the meantime. Even often bugfixing is a good \
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="#888888">--<br>
sebas<br>
<br>
<a href="http://www.kde.org" target="_blank">http://www.kde.org</a> | <a \
href="http://vizZzion.org" target="_blank">http://vizZzion.org</a> | GPG Key ID: 9119 \
0EF9<br> </font><br><br>
>> Visit <a href="http://mail.kde.org/mailman/listinfo/kde-devel#unsub" \
target="_blank">http://mail.kde.org/mailman/listinfo/kde-devel#unsub</a> to \
unsubscribe <<<br> <br></blockquote></div><br></div>
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic