<br><br><div class="gmail_quote">2011/10/6 Manuel Quiñones <span dir="ltr"><<a href="mailto:manuq@laptop.org">manuq@laptop.org</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

Thanks Gonzalo and everyone for the feedback!<br>
<br>
El día 6 de octubre de 2011 14:02, Gonzalo Odiard <<a href="mailto:gonzalo@laptop.org">gonzalo@laptop.org</a>> escribió:<br>
<div class="im">> Thanks Manuel, looks great.<br>
> A few comments about the patch:<br>
> * There are lines longer than 80 cols.<br>
> * You should not use  _("Export") + ' ' + _("As Pippy Document")<br>
> but _("Export as Pippy Document")<br>
> Nobody can assure in other languages the order will be the same.<br>
<br>
</div>You are right, I though that making like I did would not raise new<br>
translations, but I was wrong.<br>
<div class="im"><br>
> * I think we should hide the keepbutton, because is hidden in sugar 0.94 but<br>
> this activity<br>
<br>
</div>OK, done in the new patch, attached in the bug tracker.<br>
<div class="im"><br></div></blockquote><div><br></div><div>Pushed, should be in next version thanks.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div class="im">
> can be used with older Sugar versions, but this should be confirmed with the<br>
> activity maintainer.<br>
> Gonzalo<br>
<br>
</div>Thanks!<br>
<div><div></div><div class="h5"><br>
--<br>
.. manuq ..<br>
_______________________________________________<br>
Sugar-devel mailing list<br>
<a href="mailto:Sugar-devel@lists.sugarlabs.org">Sugar-devel@lists.sugarlabs.org</a><br>
<a href="http://lists.sugarlabs.org/listinfo/sugar-devel" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
</div></div></blockquote></div><br>