Fix updating of palette in sidebar

ui/sidebar.cpp

@@ -77,12 +77,14 @@ class SidebarDelegate : public QAbstractItemDelegate
         void setShowText( bool show );
         bool isTextShown() const;
 
-        void updateBrushCache();
 
         // from QAbstractItemDelegate
         void paint( QPainter *painter, const QStyleOptionViewItem &option, const QModelIndex &index ) const;
         QSize sizeHint( const QStyleOptionViewItem &option, const QModelIndex &index ) const;
 
+    private slots:
+        void updateBrushCache();
+
     private:
         bool m_showText;
         QScopedPointer<KStatefulBrush> m_windowBackground;
@@ -97,6 +99,7 @@ SidebarDelegate::SidebarDelegate( QObject *parent )
     m_selectionBackground( 0 ), m_selectionForeground( 0 )
 {
     updateBrushCache();
+    connect(qApp, &QGuiApplication::paletteChanged, this, &SidebarDelegate::updateBrushCache);
 }
 
 SidebarDelegate::~SidebarDelegate()
@@ -502,8 +505,6 @@ Sidebar::Sidebar( QWidget *parent )
     connect(d->list, &SidebarListWidget::customContextMenuRequested, this, &Sidebar::listContextMenu);
     connect(d->splitter, &QSplitter::splitterMoved, this, &Sidebar::splitterMoved);
 
-    connect(KGlobalSettings::self(), &KGlobalSettings::appearanceChanged, this, &Sidebar::appearanceChanged);
-
     setCollapsed( true );
     setFocusProxy( d->list );
 }
@@ -798,11 +799,6 @@ void Sidebar::iconSizeChanged( QAction *action )
     Okular::Settings::self()->save();
 }
 
-void Sidebar::appearanceChanged()
-{
-    d->sideDelegate->updateBrushCache();
-}
-
 void Sidebar::dragEnterEvent( QDragEnterEvent* event )
 {
     event->setAccepted( event->mimeData()->hasUrls() );

ui/sidebar.h

@@ -58,7 +58,6 @@ class Sidebar : public QWidget
         void listContextMenu( const QPoint & );
         void showTextToggled( bool );
         void iconSizeChanged( QAction *action );
-        void appearanceChanged();
 
     private:
         void setIndexEnabled( int index, bool enabled );