[Sugar-devel] Screenshot feature [GSoC Project]

[Tony Anderson]

Hi James,

Surprised that you are so involved on the Sugar side, but welcome. The 
check and x are used consistently in My Settings.

The wireless password is irrelevant to this case. The primary goal is a 
consistent dialog on quitting an activity so that the user
provides a name for an 'untitled' document or chooses not to save it. In 
this case, enter could be used to signify that the document should
be saved with the modified name and Esc to quit without saving. The 
screenshot is related as a means of providing a user-supplied name for a 
document in the Journal (the Record activivity will also require some 
extra attention since it stores documents in the Journal without the 
user having the chance to provide a name.

However, most systems provide a 'save as' dialog which uses cancel, 
save. There are two differences for Sugar. First, there is no need for 
the user to
specify the location since it is always the Journal. Second, in standard 
systems the 'save as' dialog does not quit the program. We used to have 
an option in Sugar to allow a user to save the document without quitting 
but it was removed. So in this dialog there is ambiguity with 'cancel' - 
does this mean don't save or does it mean don't quit.

This problem is further complicated by the desire in Sugar to record a 
'log' record of each activity (i.e. to store the metadata).

So the check and x may be a way to make the 'quit' dialog unambiguous 
without requiring the localization needed by save, cancel. The quit 
action could always store the metadata.

While we're at it, it might be nice to implement Esc for full screen. 
The F11 key does not work in Sugar (dedicated to sound volume).

Tony

On 04/09/2016 06:47 PM, James Cameron wrote:
> I'm sorry, but I won't be accepting the patch as it stands now.  I'm
> puzzled by your design.  Please work toward consensus before making
> such changes.
>
> Escape key handler must be put back.  Escape key must be enabled in
> all modal dialogs; as a keyboard shortcut.  It is enabled for Help, My
> Settings, View Source, and wireless password.
>
> Enter key should be put back.  I don't see why this dialog shouldn't
> accept enter key, in the same way that wireless password prompt does
> now.  Have you even tested that?
>
> The tick (check) and X icons; I don't like seeing them used in small
> dialogs; the nearest equivalent is the wireless password prompt, which
> doesn't have these icons.  But if others think the icons should be
> used, to be consistent with Help, My Settings, and View Source, then
> we should also extend it to the wireless password.
>
> @Tony, please do some more research and testing of the Escape key
> before giving such bad guidance; unless your purpose is to make these
> changes to Sugar 0.98, you must be familiar with Sugar 0.108.
>
> On Sat, Apr 09, 2016 at 04:19:00AM +0530, Utkarsh Tiwari wrote:
>> Hi,
>>       I have all these following listed inconsistencies as suggested ->
>> 1- Esc key handler removed.
>> 2- Enter key hander removed.
>> 3- Now the screenshot can be saved only via save(check) toolbutton.
>> 4- The screenshot alert can be 'not save' or 'cancelled' via cancel(X)
>> toolbutton.
>>
>> PR- https://github.com/sugarlabs/sugar/pull/675
>>
>> Please update me if further modifications are required.
>>
>> Thank you,
>> Utkarsh Tiwari
>>
>> On 4/8/16, Tony Anderson <tony_anderson at usa.net> wrote:
>>> Hi,
>>>
>>> Consistency is very important to our users. Using Esc for this is unusual.
>>>
>>>
>>> I have trouble following the screencast. It looks like you offer a check
>>> and an x for the user. It also looks like an enter after providing the
>>> title acts as
>>> the check. The x appears to save the screenshot without changing the
>>> title. The user doesn't appear to have an option not to save the
>>> screenshot. Arguably the workflow would be to complete the task and then
>>> go to the Journal to erase the unneeded screenshots. I have taken
>>> screenshots and realized, for example, that the cursor was not on the
>>> right toolbar item. For these, I would like to be able to have a 'no
>>> save' option.
>>>
>>> Using Esc is not consistent with the way the rest of Sugar works and
>>> doesn't seem needed since you offer an equivalent one-click option in
>>> the alert/pop-up.
>>>
>>> Tony
>>>
>>> On 04/08/2016 02:07 PM, Utkarsh Tiwari wrote:
>>>> Hi,
>>>>       Here is the PR to the screenshot feature ->
>>>> https://github.com/sugarlabs/sugar/pull/675
>>>>
>>>> Thank you,
>>>> Utkarsh Tiwari
>>>>
>>>> On 4/4/16, Utkarsh Tiwari <iamutkarshtiwari at gmail.com> wrote:
>>>>> Hi Dave,
>>>>>                Thanks for your appreciation. I think this feature isn't
>>>>> intrusive also because if the user wants to ignore or cancel the save,
>>>>> he can simply do so by pressing Esc key.
>>>>>
>>>>> Golden Rule- Giving the user a choice to do something but not force it.
>>>>> ;)
>>>>>
>>>>> Thanks,
>>>>> Utkarsh Tiwari
>>>>>
>>>>> On 4/4/16, Dave Crossland <dave at lab6.com> wrote:
>>>>>> I think this is a good change/feature
>>>>>>
>>>> .
>>>>
>>>