<div dir="ltr"><span style="font-family:arial,sans-serif;font-size:13px">Jérôme,</span><div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div><font face="arial, sans-serif">Here: <a href="http://sidi.mupuf.org/files/xfce4-screenshooter.tar.gz">http://sidi.mupuf.org/files/xfce4-screenshooter.tar.gz</a> are 5 individual patches for the changes I've made on my branch (except the libnotify one which I dropped). Could you please review them and tell me which can be applied to trunk?</font></div>
<div><font face="arial, sans-serif"><br></font></div><div><font face="arial, sans-serif">Thanks.</font></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-04-30 22:32 GMT+01:00 Steve Dodier-Lazaro <span dir="ltr"><<a href="mailto:sidnioulz@gmail.com" target="_blank">sidnioulz@gmail.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<div><br></div><div>I'll try and make time for a second run over Liviu's request, and the proper implementation of the text overlay. For the 1 second delay, can I then please make a config key to remove the "1 second" min delay on the spinner, and then use a usleep (min(0.2, delay) * G_USEC_PER_SEC)? This way I'd get my fast xfce4-screenshooter (and so would other power users) and the general user base would stay safe. I'm also considering adding a tiny "Settings" dialog for the filename request so that setting could be exposed over there.<br>

</div><div><br></div><div>What do you think?</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-04-30 21:08 GMT+01:00 Jérôme Guelfucci <span dir="ltr"><<a href="mailto:jeromeg@xfce.org" target="_blank">jeromeg@xfce.org</a>></span>:<div>
<div class="h5"><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
As the "current" but not active maintainer of xfce4-screenshooter,<br>
here follows my quick review:<br>
<br>
- those patches are very welcome!<br>
- don't use "//" comments, the existing code uses "/* */" comments<br>
- "Error dialog now supports markup, has nicer error message for when<br>
an…" could use being split in three (the three items in your<br>
description in fact).<br>
- "No longer waiting a whole second for a screenshot, but 200ms" I<br>
personally wouldn't accept this one. I spent hours trying to reduce<br>
this delay but it kept failing for someone. Since I increased the<br>
delay to 1s, we got 0 bug report about a crash and this delay is IMO<br>
no big deal.<br>
- "Notify the ability to cancel a region selection with Esc" I am not<br>
very found of this solution. Drawing text in the overlay window when<br>
composited or a kind of tooltip would be far better.<br>
<br>
Cheers,<br>
<br>
Jérôme<br>
<div><div><br>
On Wed, Apr 30, 2014 at 6:46 PM, Liviu Andronic <<a href="mailto:landronimirc@gmail.com" target="_blank">landronimirc@gmail.com</a>> wrote:<br>
> Dear Steve,<br>
><br>
> On Wed, Apr 30, 2014 at 1:58 AM, Steve Dodier-Lazaro<br>
> <<a href="mailto:sidnioulz@gmail.com" target="_blank">sidnioulz@gmail.com</a>> wrote:<br>
>> If you know of other (quick to write) UX improvements that are badly needed<br>
>> (and not in the bugzilla), please do tell me (except storing the Zimagez<br>
>> password in cleartext ;) ).<br>
>><br>
> One issue that absolutely needs to be fixed prior to a new release is<br>
> the naming convention:<br>
> <a href="https://bugzilla.xfce.org/show_bug.cgi?id=8445" target="_blank">https://bugzilla.xfce.org/show_bug.cgi?id=8445</a><br>
> <a href="https://bugzilla.xfce.org/show_bug.cgi?id=10348" target="_blank">https://bugzilla.xfce.org/show_bug.cgi?id=10348</a><br>
><br>
> It would be great if someone were to push a fix for this issue.<br>
><br>
> Regards,<br>
> Liviu<br>
><br>
><br>
>> Thanks,<br>
>> --<br>
>> Steve Dodier-Lazaro<br>
>> PhD Student in Information Security<br>
>> University College London<br>
>> Free Software Developer<br>
>> OpenPGP : 1B6B1670<br>
>><br>
>> _______________________________________________<br>
>> Xfce4-dev mailing list<br>
>> <a href="mailto:Xfce4-dev@xfce.org" target="_blank">Xfce4-dev@xfce.org</a><br>
>> <a href="https://mail.xfce.org/mailman/listinfo/xfce4-dev" target="_blank">https://mail.xfce.org/mailman/listinfo/xfce4-dev</a><br>
><br>
><br>
><br>
> --<br>
> Do you know how to read?<br>
> <a href="http://www.alienetworks.com/srtest.cfm" target="_blank">http://www.alienetworks.com/srtest.cfm</a><br>
> <a href="http://goodies.xfce.org/projects/applications/xfce4-dict#speed-reader" target="_blank">http://goodies.xfce.org/projects/applications/xfce4-dict#speed-reader</a><br>
> Do you know how to write?<br>
> <a href="http://garbl.home.comcast.net/~garbl/stylemanual/e.htm#e-mail" target="_blank">http://garbl.home.comcast.net/~garbl/stylemanual/e.htm#e-mail</a><br>
> _______________________________________________<br>
> Xfce4-dev mailing list<br>
> <a href="mailto:Xfce4-dev@xfce.org" target="_blank">Xfce4-dev@xfce.org</a><br>
> <a href="https://mail.xfce.org/mailman/listinfo/xfce4-dev" target="_blank">https://mail.xfce.org/mailman/listinfo/xfce4-dev</a><br>
_______________________________________________<br>
Xfce4-dev mailing list<br>
<a href="mailto:Xfce4-dev@xfce.org" target="_blank">Xfce4-dev@xfce.org</a><br>
<a href="https://mail.xfce.org/mailman/listinfo/xfce4-dev" target="_blank">https://mail.xfce.org/mailman/listinfo/xfce4-dev</a></div></div></blockquote></div></div></div><div><div class="h5"><br><br clear="all"><div><br>
</div>-- <br>Steve Dodier-Lazaro<br>
PhD Student in Information Security<br>University College London<br>Free Software Developer<br>OpenPGP : 1B6B1670
</div></div></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Steve Dodier-Lazaro<br>PhD Student in Information Security<br>University College London<br>Free Software Developer<br>OpenPGP : 1B6B1670
</div>