----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128548/ -----------------------------------------------------------
(Updated July 31, 2016, 1:26 p.m.) Status ------ This change has been marked as submitted. Review request for KDE Frameworks, David Faure and Scarlett Clark. Changes ------- Submitted with commit 7e391bbc904928f01572561482a7574ae990dce8 by Christoph Feck to branch master. Repository: kwidgetsaddons Description ------- The kdualaction autotest fails, when the icon was not found in the current theme. The reason is that KGuiItem returns a newly created Null QIcon() when passing any Null icon to setIcon(). Use a fallback icon instead. Diffs ----- autotests/kdualactiontest.cpp 6b16c89 Diff: https://git.reviewboard.kde.org/r/128548/diff/ Testing ------- Passes with all eight combinations of: - "kde" icon present/not present - platform integration present/not present - Qt 5.6.1 tag/Qt 5.7 branch Thanks, Christoph Feck
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
