> On May 26, 2015, 4:34 p.m., Martin Klapetek wrote: > > > 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.
That's correct, sorry. When I tested it, for some reason I incorrectly thought the clicked signal is emitted on button press. But there actually is now a reason to use QPushButton instead of QCommandLinkButton: QCommandLinkButton don't have vertically aligned icon and text in Breeze style ( http://wstaw.org/m/2015/05/26/qcommandlinkbutton-breeze.png this is with this commit reverted). It should be fixed in Breeze, but still... - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123546/#review80854 ----------------------------------------------------------- On May 10, 2015, 10:10 p.m., David Rosca wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/123546/ > ----------------------------------------------------------- > > (Updated May 10, 2015, 10:10 p.m.) > > > Review request for Telepathy and Martin Klapetek. > > > Repository: kaccounts-integration > > > Description > ------- > > 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). > > > Diffs > ----- > > src/accountwidget.cpp a56cf04 > src/create.cpp 5e4d46e > > Diff: https://git.reviewboard.kde.org/r/123546/diff/ > > > Testing > ------- > > Buttons are now activated on mouse release. > > > Thanks, > > David Rosca > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
