Bug 392472 - [Create new document dialogue] Put the default button on the bottom-right corner.
Summary: [Create new document dialogue] Put the default button on the bottom-right cor...
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: Usability (other bugs)
Version First Reported In: nightly build (please specify the git hash!)
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Scott Petrovic
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-03-29 03:24 UTC by Tyson Tan
Modified: 2018-04-04 13:56 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments
Examples of default button on dialogue's bottom-right corner (370.17 KB, image/png)
2018-03-29 03:24 UTC, Tyson Tan
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Tyson Tan 2018-03-29 03:24:51 UTC
Created attachment 111717 [details]
Examples of default button on dialogue's bottom-right corner

krita-4.1.0-pre-alpha-ddc6882-x86_64.appimage

Recently I noticed the default button position has been changed -- it causes a lot of mis-clicks and I think we should revert such change.

There is a reason why the default "do" button of these type of dialogues is often placed on the extreme of something -- we don't need to look and think about it -- we just go after the one of the end of the row! We open such dialogues "to do something", not so often "to cancel it". Human brain loves shortcuts. 

In the attachment I include some other dialogue examples for my argument.

This suggestion also applies for all other dialogues with similar purposes.
Comment 1 Halla Rempt 2018-03-29 07:20:36 UTC
Scott,

I'm not sure why you changed the order of the buttons. In fact, we shouldn't manage the order of ok/cancel/etc ourselves but let Qt handle that in all cases, because it follows the OS/desktop defaults.
Comment 2 Halla Rempt 2018-03-29 07:28:32 UTC
This is the class we should be using, if we're not using KoDialog which does this as well:

http://doc.qt.io/qt-5/qdialogbuttonbox.html

It might be a good project to identify all the places where we're hardcoding the dialog standard button order...
Comment 3 Halla Rempt 2018-04-03 15:18:19 UTC
Scott fixed this.
Comment 4 Tyson Tan 2018-04-04 13:56:34 UTC
Thank you!