Don't require KF5::IconThemes for Mobile

It just gives us a 48px = Large, so not worth to have that extra dependency for that in my opinion (cherry picked from commit ce8401e917)
4 years ago · bc9da32708
parent 7e248f86be
commit bc9da32708
3 changed files with 2 additions and 5 deletions
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@ -106,7 +106,6 @@ find_package(KF5 ${KF5_REQUIRED_VERSION} REQUIRED COMPONENTS
    ConfigWidgets
    CoreAddons
    I18n
-    IconThemes
    KIO
    TextWidgets
    ThreadWeaver
@ -118,7 +117,7 @@ find_package(KF5 ${KF5_REQUIRED_VERSION} REQUIRED COMPONENTS
 )

 if (BUILD_DESKTOP)
-    find_package(KF5 ${KF5_REQUIRED_VERSION} REQUIRED COMPONENTS Parts Crash)
+    find_package(KF5 ${KF5_REQUIRED_VERSION} REQUIRED COMPONENTS Parts Crash IconThemes)
 endif()

 if(KF5Wallet_FOUND)
--- a/gui/pagepainter.cpp
+++ b/gui/pagepainter.cpp
@ -7,7 +7,6 @@
 #include "pagepainter.h"

 // qt / kde includes
-#include <KIconLoader>
 #include <QApplication>
 #include <QDebug>
 #include <QIcon>
@ -33,7 +32,7 @@
 #include "settings.h"
 #include "settings_core.h"

-Q_GLOBAL_STATIC_WITH_ARGS(QPixmap, busyPixmap, (QIcon::fromTheme(QLatin1String("okular")).pixmap(KIconLoader::SizeLarge)))
+Q_GLOBAL_STATIC_WITH_ARGS(QPixmap, busyPixmap, (QIcon::fromTheme(QLatin1String("okular")).pixmap(48)))

 #define TEXTANNOTATION_ICONSIZE 24

--- a/mobile/components/CMakeLists.txt
+++ b/mobile/components/CMakeLists.txt
@ -24,7 +24,6 @@ target_link_libraries(okularplugin
        Qt5::Xml
        Qt5::Svg
        KF5::Bookmarks
-        KF5::IconThemes
        KF5::I18n
        KF5::Service
        okularcore