[frameworks-kwidgetsaddons] [Bug 391002] KPopupAccelManager::setMenuEntries() is unconditionally setting QAction::setIconText()

https://bugs.kde.org/show_bug.cgi?id=391002 Christoph Feck changed: What|Removed |Added Latest Commit||https://commits.kde.org/kwi

[frameworks-kwidgetsaddons] [Bug 391002] KPopupAccelManager::setMenuEntries() is unconditionally setting QAction::setIconText()

https://bugs.kde.org/show_bug.cgi?id=391002 --- Comment #4 from Christoph Feck --- https://phabricator.kde.org/D11346 -- You are receiving this mail because: You are watching all bug changes.

[frameworks-kwidgetsaddons] [Bug 391002] KPopupAccelManager::setMenuEntries() is unconditionally setting QAction::setIconText()

https://bugs.kde.org/show_bug.cgi?id=391002 --- Comment #3 from Christian Ehrlicher --- When you set the iconText only when the text is changed it would at least fix the bug report I would guess. The problem is that it's unknown if the iconText is set programatically or not and even watching for

[frameworks-kwidgetsaddons] [Bug 391002] KPopupAccelManager::setMenuEntries() is unconditionally setting QAction::setIconText()

https://bugs.kde.org/show_bug.cgi?id=391002 --- Comment #2 from Christoph Feck --- For reference: https://cgit.kde.org/kwidgetsaddons.git/tree/src/kacceleratormanager.cpp#n793 -- You are receiving this mail because: You are watching all bug changes.

[frameworks-kwidgetsaddons] [Bug 391002] KPopupAccelManager::setMenuEntries() is unconditionally setting QAction::setIconText()

https://bugs.kde.org/show_bug.cgi?id=391002 --- Comment #1 from Christoph Feck --- Thanks Christian for the investigation. Do I understand it correctly, that moving the setIconText() call into the 'if' (after line 793) is sufficient to fix the issue? -- You are receiving this mail because: You