D7196: Fix accepting dialogs with ctrl+return when buttons are renamed

2017-08-09 Thread Aleix Pol Gonzalez
This revision was automatically updated to reflect the committed changes. Closed by commit R252:7524bf1535b4: Fix accepting dialogs with ctrl+return when buttons are renamed (authored by apol). REPOSITORY R252 Framework Integration CHANGES SINCE LAST UPDATE

D7196: Fix accepting dialogs with ctrl+return when buttons are renamed

2017-08-09 Thread David Faure
dfaure accepted this revision. This revision is now accepted and ready to land. REPOSITORY R252 Framework Integration BRANCH master REVISION DETAIL https://phabricator.kde.org/D7196 To: apol, #frameworks, dfaure

D7196: Fix accepting dialogs with ctrl+return when buttons are renamed

2017-08-07 Thread Aleix Pol Gonzalez
apol updated this revision to Diff 17862. apol added a comment. Make sure there's such a button REPOSITORY R252 Framework Integration CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D7196?vs=17860=17862 BRANCH master REVISION DETAIL https://phabricator.kde.org/D7196 AFFECTED

D7196: Fix accepting dialogs with ctrl+return when buttons are renamed

2017-08-07 Thread Aleix Pol Gonzalez
apol created this revision. Restricted Application added a project: Frameworks. REVISION SUMMARY QAbstractButton::setText will call setShortcut(mnemonic), making this little hack unusable. This patch puts the shortcut into a separate object. TEST PLAN Improved the test to reproduce the