Commit 2042e4da authored by René J.V. Bertin's avatar René J.V. Bertin
Browse files

Don't remove an existing app icon

This corrects commit 0d56bd9a
which had the effect of removing the embedded application icon
on systems like Mac and MS Windows where icon themes are typically
not configured, and QIcon::fromTheme() thus returns an empty QIcon.
QIcon::fromTheme() should always be called with a valid fallback
icon, for instance the target icon (QApplication::windowIcon()).
parent ad0b96a7
......@@ -405,7 +405,8 @@ int main( int argc, char *argv[] )
KAboutData::setApplicationData(aboutData);
// set icon for shells which do not use desktop file metadata
QApplication::setWindowIcon(QIcon::fromTheme(QStringLiteral("kdevelop")));
// but without setting replacing an existing icon with an empty one!
QApplication::setWindowIcon(QIcon::fromTheme(QStringLiteral("kdevelop"), QApplication::windowIcon()));
KCrash::initialize();
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment