<div dir="ltr"><div>Done! See:<br><a href="https://git.reviewboard.kde.org/r/111849/">https://git.reviewboard.kde.org/r/111849/</a><br><br></div>Thank you very much!<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">
2013/8/2 Sebastian Kügler <span dir="ltr"><<a href="mailto:sebas@kde.org" target="_blank">sebas@kde.org</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Hu Zheng,<br>
<br>
Thanks for the patch, and welcome to Plasma development!<br>
<div class="im"><br>
On Friday, August 02, 2013 20:01:19 Hu Zheng wrote:<br>
> Here is the custom first week patch. Can anyone add it to the source<br>
> repository?<br>
<br>
</div>We use reviewboard to review changes to Plasma source code. This way, we can<br>
look at the changes in detail, improve things that need fixing, and review for<br>
example changes in the UI.<br>
<br>
Could you post your patch to our Reviewboard instance at<br>
<a href="http://git.reviewboard.kde.org" target="_blank">http://git.reviewboard.kde.org</a> and assign it to the Plasma group?<br>
<br>
A good explanation of what your code tries to achieve and how it does that<br>
helps reviewers to understand the changes.<br>
As your code introduces visual changes, please also attach screenshots (before<br>
and after) to the review request.<br>
<br>
Also, a note on our development cycle. kde-workspace is feature-frozen, as we<br>
move to development of the upcoming version of Plasma. This means that any new<br>
feature has to target Plasma Workspaces 2, which we intend to release next<br>
summer.<br>
<br>
Kind regards,<br>
<div class="HOEnZb"><div class="h5">--<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>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><a href="http://www.huzheng.org">http://www.huzheng.org</a>
</div>