<br><br><div><span class="gmail_quote">2007/11/28, Jan Kundrát <<a href="mailto:jkt@gentoo.org">jkt@gentoo.org</a>>:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

Paul Fleischer wrote: > As i wrote in a comment to Bug 152438, I have created a very > experimental system for grouping images in KPhotoAlbum. Wow, looks good.</blockquote><div> I didn't have a look yet. I'll try to try in the next days :-).

<br></div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Some comments that I have at a first glance:<br><br>b) You've used another whitespace convention, you should've used 4

<br>spaces per indent</blockquote><div><br>Surely I'm out of the way, moreover I didn't look at the code about it. <br>But shouldn't this indentation size (done by spaces if I understand correctly) be put in some global parameter ? 

<br>This way we could later consider to allow defining new size for this identation ?<br></div></div><br>Cheers.<br><br>-- <br>Baptiste <Batmat> MATHUS<br>BMathus at Batmat point net - <a href="http://batmat.net">http://batmat.net

</a> Si chacun de nous a une idée et que nous les partageons, nous repartirons tous les deux avec deux idées... C'est ça le Libre.