Fix shortcut saving on KDE, and use the system dialog for Frameworks
authorManuel Nickschas <sputnick@quassel-irc.org>
Mon, 16 Feb 2015 20:56:48 +0000 (21:56 +0100)
committerManuel Nickschas <sputnick@quassel-irc.org>
Mon, 16 Feb 2015 20:56:48 +0000 (21:56 +0100)
commit56f70df273886f020699d296bcb7e3fbea237c09
tree571a491ed5b1b9a83d3c3cc603b87240ca1bc2e3
parent446caa053a85a00ef1f2ed393b5a0da761d6f2d1
Fix shortcut saving on KDE, and use the system dialog for Frameworks

While re-enabling the system dialog for shortcut editing for KF5,
I finally figured out why shortcuts where never saved when using KDE;
a long standing bug we've had.
Turns out QDialog::exec() does not actually save things. All this time
I was looking into completely different directions; but it never
occured to me that I need to explicitly save after the dialog is
accepted (our own dialog does this automatically). This is not a
complaint; it was being stupid on my end :)

Fixes #1057.
src/qtui/mainwin.cpp
src/uisupport/CMakeLists.txt
src/uisupport/actioncollection.cpp
src/uisupport/actioncollection.h