-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126184/
-----------------------------------------------------------
Review request for KDE Frameworks and David Faure.
Bugs: 352882
https://bugs.kde.org/show_bug.cgi?id=352882
Repository: kxmlgui
Description
-------
To Reproduce: in a KXmlGuiWindow, open the Configure Toolbars dialog.
1 - Change the main toolbar setting.
2 - Click Apply, the ui is rebuild, and the newToolBarConfig signal is emitted,
allowing main app to re-plug its dynamic actions
3 - Click Ok, the ui is rebuild but the signal is not emitted, so app cannot
re-plug actions, and they are lost.
The problem only happens if you click "apply" and then "ok". This is due to the
fact that when Ok is clicked, the ui is rebuild (m_widget->save triggers ui
rebuild), but signal is not emitted.
My solution is to rebuild ui only if something changed when clicking "ok".
Diffs
-----
src/kedittoolbar.cpp ba4ba0f
Diff: https://git.reviewboard.kde.org/r/126184/diff/
Testing
-------
Tested, fixes the reported problem.
Thanks,
Jean-Baptiste Mardelle
_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
