Some UX changes to Screenshooter

[Jérôme Guelfucci]

Hi,

As the "current" but not active maintainer of xfce4-screenshooter,
here follows my quick review:

- those patches are very welcome!
- don't use "//" comments, the existing code uses "/* */" comments
- "Error dialog now supports markup, has nicer error message for when
an…" could use being split in three (the three items in your
description in fact).
- "No longer waiting a whole second for a screenshot, but 200ms" I
personally wouldn't accept this one. I spent hours trying to reduce
this delay but it kept failing for someone. Since I increased the
delay to 1s, we got 0 bug report about a crash and this delay is IMO
no big deal.
- "Notify the ability to cancel a region selection with Esc" I am not
very found of this solution. Drawing text in the overlay window when
composited or a kind of tooltip would be far better.

Cheers,

Jérôme

On Wed, Apr 30, 2014 at 6:46 PM, Liviu Andronic <landronimirc at gmail.com> wrote:
> Dear Steve,
>
> On Wed, Apr 30, 2014 at 1:58 AM, Steve Dodier-Lazaro
> <sidnioulz at gmail.com> wrote:
>> If you know of other (quick to write) UX improvements that are badly needed
>> (and not in the bugzilla), please do tell me (except storing the Zimagez
>> password in cleartext ;) ).
>>
> One issue that absolutely needs to be fixed prior to a new release is
> the naming convention:
> https://bugzilla.xfce.org/show_bug.cgi?id=8445
> https://bugzilla.xfce.org/show_bug.cgi?id=10348
>
> It would be great if someone were to push a fix for this issue.
>
> Regards,
> Liviu
>
>
>> Thanks,
>> --
>> Steve Dodier-Lazaro
>> PhD Student in Information Security
>> University College London
>> Free Software Developer
>> OpenPGP : 1B6B1670
>>
>> _______________________________________________
>> Xfce4-dev mailing list
>> Xfce4-dev at xfce.org
>> https://mail.xfce.org/mailman/listinfo/xfce4-dev
>
>
>
> --
> Do you know how to read?
> http://www.alienetworks.com/srtest.cfm
> http://goodies.xfce.org/projects/applications/xfce4-dict#speed-reader
> Do you know how to write?
> http://garbl.home.comcast.net/~garbl/stylemanual/e.htm#e-mail
> _______________________________________________
> Xfce4-dev mailing list
> Xfce4-dev at xfce.org
> https://mail.xfce.org/mailman/listinfo/xfce4-dev