This would be fine with me. There is almost a month to feature freeze and only two of the features originally agreed are left to land, with one having had already an initial review. So I'd say we have enough time to handle this.<div>
<br></div><div>Manuel, Simon, what do you think?<span></span></div><div><div><div><div><br>On Friday, 5 July 2013, Gonzalo Odiard  wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">As the time to include new features has ended,<div>I want request a exception to include one more change.</div><div><br></div><div>This feature modify the ObjectChooser, adding a option show_preview,</div><div>

if the option is True, instead of showing a listview similar to the Journal,</div><div>show a IconView with the object preview, and the object title.</div><div><br></div><div>This is useful for activities with a option to include images from the Journal,</div>

<div>as Paint, Write, Fototoon, and Memorize.</div><div><br></div><div>I added the option instead of adding the iconview to all the ObjectChooser,</div><div>because is a bit slower to start if you have a Journal with many images.</div>

<div><br></div><div>Regarding implementation, there are changes in sugar [1] and sugar-toolkit-gtk3 [2]</div><div>Every branch have two patches, the first is a preparation, moving the code used </div><div>to generate a image from the datastore preview data, from expandedentry.py in sugar,</div>

<div>to datastore.py in sugar-toolkit-gtk3. I moved this to the toolkit,</div><div>because there are activities using that too, like Portfolio and JournalShare.</div><div>May be should be better create another package as datastore/utils or similar</div>

<div>instead of using datastore/datastore, because I couldn't use the size constants</div><div>defined in activity.py, because activity.py already import datastore/datastore.</div><div><br></div><div>The other change implement a iconmodel/iconview, similar to listmodel/listview,</div>

<div>using a custom CellRendererPIxbuf to genereate the pixbuf as needed,</div><div>instead of generate all the images at once. In the future we can show a star</div><div>for the favorite objects using the same renderer.  </div>

<div><br></div><div>Comments are welcome,</div><div><br></div><div>Gonzalo</div><div><br></div><div>[1] <a href="https://github.com/sugarlabs/sugar/pull/48" target="_blank">https://github.com/sugarlabs/sugar/pull/48</a></div>
<div>[2] <a href="https://github.com/sugarlabs/sugar-toolkit-gtk3/pull/17" target="_blank">https://github.com/sugarlabs/sugar-toolkit-gtk3/pull/17</a> </div>
</div>
</blockquote></div></div></div></div><br><br>-- <br>Daniel Narvaez<br><br>