-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129885/
-----------------------------------------------------------

(Updated Ян. 25, 2017, 10:39 след обяд)


Review request for KDE Frameworks, Albert Astals Cid and David Faure.


Summary (updated)
-----------------

[KStandadAction] Correct menubars logic


Repository: kconfigwidgets


Description (updated)
-------

This patch (https://git.reviewboard.kde.org/r/129261/) is *ugly* makes side 
effects on KDE menubars. Start Dolphin -> hide menubar -> restart it -> click 
control button -> menubar action is checked.
It hide all actions changed between updateAction calls e.g.
updateAction (stores last state of checked and enable)
application changed menu action like it want
updateAction is called on show event -> discard modified states


Diffs
-----

  src/kstandardaction.cpp 6e5bba7 

Diff: https://git.reviewboard.kde.org/r/129885/diff/


Testing
-------

Works correct on KDE, test it on Unity.
Please don't make such a patch, respect application behavior.


Thanks,

Anthony Fieroni

Reply via email to