<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">2014-04-17 20:04 GMT+02:00 Ingo Klöcker <span dir="ltr"><<a href="mailto:kloecker@kde.org" target="_blank">kloecker@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">
<div class="">On Wednesday 16 April 2014 12:56:01 Elvis Angelaccio wrote:<br>
> So, the choice is between the branch <test> and <test2>. Let me know<br>
> what do you prefer.<br>
><br>
> For completeness the alternatives in details are:<br>
><br>
> - branch <test>: no splitters, with dividers (i.e. 4 QFrames in a<br>
> horizontal layout):<br>
> <a href="http://abload.de/img/kronometer-no-splittevdklc.png" target="_blank">http://abload.de/img/kronometer-no-splittevdklc.png</a><br>
<br>
</div>This would look much better if you remove the ':' and '.' after the<br>
numbers. IMO the ':'/'.' are superfluous because the numbers are already<br>
clearly separated by the frame border.<br>
<br>
Also you should probably right-align the numbers. In particular, the<br>
hours. For the other numbers alignment probably doesn't matter.<br>
Moreover, I think it would look best if all four frames were the same<br>
size.<br>
<div class=""><br></div></blockquote><div><div>Hi, thanks for your suggestions!</div><div>I've committed all these changes on the <test> branch. </div><div>I've applied the right alignment to all the labels because looks more consistent. (e.g when the window is full size).</div>
</div><div> <br></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>
> - branch <test2>:<br>
> no splitters, no dividers (i.e. single QFrame with a grid layout):<br>
> <a href="http://abload.de/img/kronometer-no-divideri4kce.png" target="_blank">http://abload.de/img/kronometer-no-divideri4kce.png</a><br>
<br>
</div>Here the ':'/'.' need to stay for obvious reasons. But you should try to<br>
make the spacing between the numbers and the ':'/'.' identical. Possible<br>
solution: Put the ':'/'.' into columns of their own. And right-align the<br>
hours.<br>
<br></blockquote><div><div><br></div><div><div>This one is not trivial. I could use columns on their own for the "dividers", but these symbols are displayed only if there are numbers on their right.</div><div>At the moment I handle this check in the QTimeFormat class, it would be difficult to move this logic to a widget class like QTimeDisplay is.</div>
<div><br></div><div>Since also Albert agrees with the first alternative, probably the second one it's not worth of the effort.</div></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">

<br>
I'd also get rid of the upper toolbar. You do already have the essential<br>
tools in the lower toolbar and having two toolbars even with differently<br>
sized icons makes the UI look unnecessarily crowded.<br>
<br></blockquote><div><br></div><div>Final screenshot with all these changes: <a href="http://abload.de/img/kronometerdrkjq.png">http://abload.de/img/kronometerdrkjq.png</a></div><div><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">

<br>
Just my two cents.<br>
<br>
<br>
Regards,<br>
Ingo<br></blockquote><div><br></div><div>Regards,</div><div>Elvis </div></div><br></div></div>