<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">2014-04-14 1:06 GMT+02:00 Albert Astals Cid <span dir="ltr"><<a href="mailto:aacid@kde.org" target="_blank">aacid@kde.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
El Dilluns, 7 d'abril de 2014, a les 23:52:19, Elvis Angelaccio va escriure:<br>
> Hi all,<br>
<br>
Hi!<br>
<div class=""><br></div></blockquote><div> </div><div>Hi Albert,</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div class="">
> with this email I'm going to ask a review for Kronometer, in order to be<br>
> accepted in KDE.<br>
> Kronometer is a stopwatch application for KDE. It's meant to be simple but<br>
> also customizable.<br>
><br>
> Kronometer has been moved to kdereview from its previous location,<br>
> playground/utils. I'm not sure whether to ask the admission in<br>
> extragear-utils or in kdeutils.<br>
<br>
</div>Personally I don't see it being a broad enough use case to make sense to be in<br>
kdeutils. What do others think? Have you asked at kde-utils-devel<br>
<a href="https://mail.kde.org/mailman/listinfo/kde-utils-devel" target="_blank">https://mail.kde.org/mailman/listinfo/kde-utils-devel</a> ?<br></blockquote><div><br></div><div>No, I have not yet asked. I will do it.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div class=""><br>
> What I'm looking for is the help of the KDE community with translations,<br>
> packaging and bug-tracking. If the choice is definitely up to me, it would<br>
> be nice to join the kdeutils module.<br>
><br>
> Regarding the requirements for the admission:<br>
><br>
> 1. There is the documentation in DocBook format. Thanks to Yuri Chornoivan<br>
> for his help.<br>
> 2. Source code is documented using the doxygen syntax, as suggested in the<br>
> techbase documentation policy.<br>
> 3. All the krazy code checker issues have been addressed.<br>
> 4. No usability review has been done, but it's welcome.<br>
> 5. Profiler: unfortunately I don't know how to do it. I used Valgrind and<br>
> there shouldn't be memory leaks.<br>
> I tried to use also Callgrind but I'm not able to understand its output. If<br>
> a profiler check is strictly required, I'll need help for it.<br>
<br>
</div>Nah, it's not like your app is doing anything very resource intensive so you<br>
don't need performance testing (just make sure you don't hog the cpu at 100%<br>
:D)<br>
<br>
Some small comment from my side:<br>
 * You are passing an email address as bug address, you should leave the<br>
default bugzilla one there and create a kronometer bug entry in <a href="http://bugs.kde.org" target="_blank">bugs.kde.org</a><br>
if you don't have power for that ask to the sysadmin guys about it.<br></blockquote><div><br></div><div>Yes, I put only a temporary email address, waiting for an official bugs entry.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

 * Your choice of splitters to separate hours/minutes/seconds seems a bit<br>
weird do you think that anyone will use it to have something like very wide<br>
minutes and narrow the rest?<br></blockquote><div><br></div><div><div>I see your point, probably the splitters are unnecessary UI components for this use case.</div><div>What do you think about an option in "Interface settings"? I could display by default a single QFrame (without splitters) and leave to the user an opt-in to allow the splitters.</div>
<div>In this way I can reuse the existing code without too much refactoring.</div></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

 * The general/font/save settings probably would look nicer with a vertical<br>
spacer at the end that eats up empty space when the vertical space is bigger<br>
than needed (i.e. similar to what you have in interface settings).<br></blockquote><div><br></div><div>Good catch, there the spacers have been forgotten.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<br>
Cheers,<br>
  Albert<br>
<div class=""><br>
<br></div></blockquote><div><br></div><div>Regards,<br>Elvis</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div class="">
> 6. The application should be completely translatable, thanks again to the<br>
> help of Yuri.<br>
><br>
> Finally here the references:<br>
><br>
> Kronometer repository in kdereview:<br>
</div>> *<a href="https://projects.kde.org/projects/kdereview/kronometer" target="_blank">https://projects.kde.org/projects/kdereview/kronometer</a><br>
> <<a href="https://projects.kde.org/projects/kdereview/kronometer" target="_blank">https://projects.kde.org/projects/kdereview/kronometer</a>>*<br>
<div class="">> Kronometer quickgit: <a href="http://quickgit.kde.org/?p=kronometer.git" target="_blank">http://quickgit.kde.org/?p=kronometer.git</a><br>
> Kronometer website:<br>
</div>> <a href="http://aelog.org/kronometer/" target="_blank">http://aelog.org/kronometer/</a><<a href="http://www.aelog.org/kronometer/" target="_blank">http://www.aelog.org/kronometer/</a>><br>
<div class=""><div class="h5">><br>
> If you want to quickly browse the code, you can also do it whit the Woboq's<br>
> code browser here: <a href="http://aelog.org/codebrowser/kronometer/" target="_blank">http://aelog.org/codebrowser/kronometer/</a><br>
><br>
> Thank you for your time,<br>
> Elvis Angelaccio<br>
<br>
</div></div></blockquote></div><br></div></div>