[pve-devel] [PATCH manager 0/5] Usability and Esthetics improvement

[Emmanuel Kasper]

On 07/10/2017 11:19 AM, Dominik Csapak wrote:
> On 07/06/2017 02:01 PM, Emmanuel Kasper wrote:
>> This patch serie tries to iron out some rough aspects of the UI.
>> From the usability perspective the first one is the most important IMHO
>>
>> Emmanuel Kasper (5):
>>   Add a "Run task in background"  button to make users feel safer
>>   Do not display the abbreviated commit SHA-1 in the workspace version
>>     string
>>   Close #1268: Show migration start server and migration end server
>>   Rename the pve-help-button to pve-inline-button
>>   Improve a bit the esthetics of the User Settings window
>>
>>  www/css/ext6-pve.css              |  2 +-
>>  www/manager6/Workspace.js         |  3 +--
>>  www/manager6/button/HelpButton.js |  2 +-
>>  www/manager6/window/Migrate.js    |  6 ++++--
>>  www/manager6/window/Settings.js   |  7 +++----
>>  www/manager6/window/TaskViewer.js | 15 +++++++++++++--
>>  6 files changed, 23 insertions(+), 12 deletions(-)
>>
> 
> looks good, but one remark on 1/5 Add a "Run task in background"  button
> to make users feel safer :
> 
> i guess this would confuse those users, who already clicked the 'X'
> and know it will continue the task.
> they probably will now think that the 'X' cancels/stops the task
> which is not what we want them to think
> 
> i would instead propose trying to use a '-' in the corner,
> to symbolize only 'minimizing' instead of closing

I tried this approach, unfortunately it looks even more surprising
compared to the run in background button approach.
see this http://imgur.com/a/PoL0P

Also this patch changes the aspect of the run in background button after
the containing task is completed, providing more visual feedback,
something which is not possible with the window using the standard ExtJS
Window properties (minimizable/closable)