Some UX changes to Screenshooter
jeromeg at xfce.org
Wed Apr 30 22:08:21 CEST 2014
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.
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:
> It would be great if someone were to push a fix for this issue.
>> 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
> Do you know how to read?
> Do you know how to write?
> Xfce4-dev mailing list
> Xfce4-dev at xfce.org
More information about the Xfce4-dev