<br><br><div><span class="gmail_quote">2007/5/16, 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;">
Hi,<br>I have some patches that I'd like to apply. As per Jesper's suggestions,<br>I'd like to hear a yes/no from mother people before I commit them and<br>break stuff.<br><br><br>Two trivial patches for extending the list of supported files:
<br><br><a href="http://dev.gentoo.org/~jkt/tmp/bug-141877-ImageManager-recognize-erf.patch">http://dev.gentoo.org/~jkt/tmp/bug-141877-ImageManager-recognize-erf.patch</a><br><a href="http://dev.gentoo.org/~jkt/tmp/bug-145366-video-flv.patch">
http://dev.gentoo.org/~jkt/tmp/bug-145366-video-flv.patch</a></blockquote><div><br>OK for those two. I don't see any side effect it could have :-).<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;">
Make "show this image" launch viewer with *all* images form current view<br>and seek to the selected one.<br><br>NOTE: this will slow down viewing single image when having a lot of<br>pictures displayed (not a typical situation, I guess).
<br><br><a href="http://dev.gentoo.org/~jkt/tmp/bug-145309-show-this-image-shows-all.patch">http://dev.gentoo.org/~jkt/tmp/bug-145309-show-this-image-shows-all.patch</a></blockquote><div><br>I mainly reviewed this one from a user perspective. I looked a bit at the code, apart from a typo (appers instead of appears I guess) the rest seems correct.
<br><br>I don't know KPA architecture enough to know if we'd have some way to have a more direct access to the picture actually selected, something to avoid scrolling to the current one. <br><br>O(n) complexity in this case would be interesting to test with a 20000 photo db or more. If I select the last one on my 6000pix db, it's not really annoying.
<br><br>So, OK for me for this patch. At least the feature is really interesting, and we will still be able to look for a way to improve it if necessary, isn't it?<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;">
Annotation dialog: add button that copies tags from "previously tagged"<br>image (see bugreport for details):<br><br>NOTE: this also includes a fix for bug 136374. I'm not sure if I can<br>touch the i18n() strings. Feel free to change the wording.
<br><br><a href="http://dev.gentoo.org/~jkt/tmp/bug-145158-retag-with-tags-from-previous-image.patch">http://dev.gentoo.org/~jkt/tmp/bug-145158-retag-with-tags-from-previous-image.patch</a></blockquote><div><br>Haven't reviewed this one yet. So, I won't vote for the moment. Maybe tomorrow if I find time.
<br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Thanks in advance,</blockquote><div><br>Thanks to *you* for your work :-)<br></div>
</div><br clear="all"><br>-- <br>Baptiste <Batmat> MATHUS<br>BMathus at Batmat point net - <a href="http://batmat.net">http://batmat.net</a><br>---------<br>Si chacun de nous a une idée et que nous les partageons, nous
<br>repartirons tous les deux avec deux idées... C'est ça le Libre.