Fix crash in mouse click config
If an object is QObject parent managed it should not have any other memory management system.
If we want member variables to not have a raw pointer the best approach is to leave the UI class on the stack as it is now and only create a widget where we need it which can remain owned by the QObject parent.
We need to fix all the KCMs, if this is acked in principle I'll go do that.