<div class="gmail_quote">On Sat, Apr 18, 2009 at 2:58 AM, Andrea Diamantini <span dir="ltr"><<a href="mailto:adjam7@gmail.com">adjam7@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
I wanna start this mail thanking all you again for your fantastic work<br>
and help to rekonq project.<br>
Questions about the no merging issues during these holidays are basically two<br>
I'm seeing in your clones.<br>
Clones are actually rekonq forks and as this always helps features and ideas<br>
to be implemented lets discerning quite hard.<br>
<br>
So here are some small rules to follow to help at best rekonq dev:<br>
<br>
1.<br>
NO large commits. as avaddon just explained in one post some days ago (i read<br>
all just this morning), we have one fantastic rule:<br>
<br>
ONE feature <= ONE commit.</blockquote><div><br></div><div>Yes this is very good to have small easy to understand commits, that's what I'm trying to learn myself to make coding together nice and smooth experience, but sometimes this isn't simple, when one bug is triggered by another or stuff like that.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
So all we can push one feature with one or more commits, but one commit has to<br>
be about just 1 feature.<br>
<br>
<br>
2.<br>
NO features non approved in master. For example I saw a beautiful side panel<br>
implementation. But rekonq will never have a side panel. That's because side<br>
panels are usability evils and I'd like rekonq to be "easy".</blockquote><div><br></div><div><span class="Apple-style-span" style="font-family: -webkit-sans-serif; line-height: 19px; ">"Never say never", actually now it have ;P Maybe they are usability nightmare for you but not for everyone.</span></div>
<div><font class="Apple-style-span" face="-webkit-sans-serif"><span class="Apple-style-span" style="line-height: 19px;">I think side panel is basic feature, all browsers have one and I think it should be implemented in modular way.</span></font></div>
<div><font class="Apple-style-span" face="-webkit-sans-serif"><span class="Apple-style-span" style="line-height: 19px;">I was thinking about moving all side panel code to shared library, so users who don't want to use it won't have to pay for it.</span></font></div>
<div><font class="Apple-style-span" face="-webkit-sans-serif"><span class="Apple-style-span" style="line-height: 19px;"><br></span></font></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
3.<br>
NO unuseful re-implementations. I really spent one hour to review pawel<br>
Download class refactoring, that was able just to introduce just one bug<br>
(issue #?) and no more features. Why do we feel the needing of doing such<br>
things?? And just to be honest, that class is copied from kget code.</blockquote><div> </div><div>I don't fully understand what do you mean by "copied from kget code" because I've never seen kget's code.</div>
<div>Only because you don't understand why some changes were made doesn't mean they are useless.</div><div>All refactoring was done in the sake of maintainability and some new features were added in process.</div>
<div>As well as making download actually work instead of "work sort off" and crash on background download.</div><div><br></div><div>"Download class refactoring, that was able just to introduce just one bug</div>
(issue #?) and no more features." - what commit introduced which bug?</div><div class="gmail_quote"><br><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
I used side panel and download class just as examples about my ideas. You are<br>
obviously free to do as you like, but please, do it in separate branches and<br>
push masters nearer.<br>
<br>
--<br>
Andrea Diamantini<br>
MAIL: adjam7_AT_gmail_DOT_com<br>
WEB: <a href="http://www.adjam.org" target="_blank">http://www.adjam.org</a><br>
IRC: adjam_AT_freenode<br>
PGP/GPG : 91A712C1<br>
Fingerprint: 571E DFF4 19EF A597 2CCD A811 6CB6 3538 91A7 12C1<br>
<br>
tadarattadara tattà tatatatatà tadarattadara tattà tattattattattà..<br>
(me, taking a shower...)<br>
_______________________________________________<br>
rekonq mailing list<br>
<a href="mailto:rekonq@kde.org">rekonq@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/rekonq" target="_blank">https://mail.kde.org/mailman/listinfo/rekonq</a><br>
</blockquote></div><br>