Show dialogs on button clicked (not pressed)
Review Request #123546 - Created April 28, 2015 and submitted
Connect to QPushButton::clicked signal instead of QPushButton::pressed.
This makes it consistent with other buttons, eg. the button action is activated only when releasing the mouse inside the button rect, not immediately on mouse press.
It is also needed to change QCommandLinkButton to QPushButton, where it changes the text alignment from left-aligned to center-aligned. I don't know how to replicate the look with
QPushButton, but we should not use QCommandLinkButton as it is activated on press (and there is no way to easily change it).
Buttons are now activated on mouse release.
Status: Closed (submitted)
Submitted with commit 47734ffd90bdd40350379ffecf7f055fb0eec9a8 by David Rosca to branch master.
but we should not use QCommandLinkButton as it is activated on press (and there is no way to easily change it).
QCommandLinkButton derives from QPushButton, it has exactly the same signals including clicked(); there's really no reason to change that. I'll be changing it back to QCommandLinkButton.