diff --git a/src/kpageview.cpp b/src/kpageview.cpp
index 10ad3c7..d230ea0 100644
--- a/src/kpageview.cpp
+++ b/src/kpageview.cpp
@@ -1,569 +1,564 @@
 /*
     This file is part of the KDE Libraries
 
     Copyright (C) 2006 Tobias Koenig (tokoe@kde.org)
     Copyright (C) 2007 Rafael Fernández López (ereslibre@kde.org)
 
     This library is free software; you can redistribute it and/or
     modify it under the terms of the GNU Library General Public
     License as published by the Free Software Foundation; either
     version 2 of the License, or (at your option) any later version.
 
     This library is distributed in the hope that it will be useful,
     but WITHOUT ANY WARRANTY; without even the implied warranty of
     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
     Library General Public License for more details.
 
     You should have received a copy of the GNU Library General Public License
     along with this library; see the file COPYING.LIB. If not, write to
     the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
     Boston, MA 02110-1301, USA.
 */
 
 #include "kpageview.h"
 #include "kpageview_p.h"
 
 #include "kpagemodel.h"
 #include "loggingcategory.h"
 
 #include <ktitlewidget.h>
 
 #include <QAbstractItemView>
 #include <QGridLayout>
 #include <QLabel>
 #include <QSize>
 #include <QStackedWidget>
 #include <QTimer>
 
 void KPageViewPrivate::_k_rebuildGui()
 {
     // clean up old view
     Q_Q(KPageView);
 
     QModelIndex currentLastIndex;
     if (view && view->selectionModel()) {
         QObject::disconnect(view->selectionModel(), SIGNAL(selectionChanged(QItemSelection,QItemSelection)),
                             q, SLOT(_k_pageSelected(QItemSelection,QItemSelection)));
         currentLastIndex = view->selectionModel()->currentIndex();
     }
 
     delete view;
     view = q->createView();
 
     Q_ASSERT(view);
 
     view->setSelectionBehavior(QAbstractItemView::SelectItems);
     view->setSelectionMode(QAbstractItemView::SingleSelection);
 
     if (model) {
         view->setModel(model);
     }
 
     // setup new view
     if (view->selectionModel()) {
         QObject::connect(view->selectionModel(), SIGNAL(selectionChanged(QItemSelection,QItemSelection)), q, SLOT(_k_pageSelected(QItemSelection,QItemSelection)));
 
         if (currentLastIndex.isValid()) {
             view->selectionModel()->setCurrentIndex(currentLastIndex, QItemSelectionModel::Select);
         } else if (model) {
             view->selectionModel()->setCurrentIndex(model->index(0, 0), QItemSelectionModel::Select);
         }
     }
 
     if (faceType == KPageView::Tabbed) {
         stack->setVisible(false);
         layout->removeWidget(stack);
     } else {
         layout->addWidget(stack, 2, 1);
         stack->setVisible(true);
     }
 
     if (pageHeader) {
         pageHeader->setVisible(q->showPageHeader());
         titleWidget->setVisible(false);
     } else {
         titleWidget->setVisible(q->showPageHeader());
     }
 
     Qt::Alignment alignment = q->viewPosition();
     if (alignment & Qt::AlignTop) {
         layout->addWidget(view, 2, 1);
     } else if (alignment & Qt::AlignRight) {
         layout->addWidget(view, 1, 2, 2, 1);
     } else if (alignment & Qt::AlignBottom) {
         layout->addWidget(view, 4, 1);
     } else if (alignment & Qt::AlignLeft) {
         layout->addWidget(view, 1, 0, 3, 1);
     }
 }
 
 void KPageViewPrivate::updateSelection()
 {
     /**
      * Select the first item in the view if not done yet.
      */
 
     if (!model) {
         return;
     }
 
     if (!view || !view->selectionModel()) {
         return;
     }
 
     const QModelIndex index = view->selectionModel()->currentIndex();
     if (!index.isValid()) {
         view->selectionModel()->setCurrentIndex(model->index(0, 0), QItemSelectionModel::Select);
     }
 }
 
 void KPageViewPrivate::cleanupPages()
 {
     /**
      * Remove all orphan pages from the stacked widget.
      */
 
     const QList<QWidget *> widgets = collectPages();
 
     for (int i = 0; i < stack->count(); ++i) {
         QWidget *page = stack->widget(i);
 
         bool found = false;
         for (int j = 0; j < widgets.count(); ++j) {
             if (widgets[ j ] == page) {
                 found = true;
             }
         }
 
         if (!found) {
             stack->removeWidget(page);
         }
     }
 }
 
 QList<QWidget *> KPageViewPrivate::collectPages(const QModelIndex &parentIndex)
 {
     /**
      * Traverse through the model recursive and collect all widgets in
      * a list.
      */
     QList<QWidget *> retval;
 
     int rows = model->rowCount(parentIndex);
     for (int j = 0; j < rows; ++j) {
         const QModelIndex index = model->index(j, 0, parentIndex);
         retval.append(qvariant_cast<QWidget *>(model->data(index, KPageModel::WidgetRole)));
 
         if (model->rowCount(index) > 0) {
             retval += collectPages(index);
         }
     }
 
     return retval;
 }
 
 KPageView::FaceType KPageViewPrivate::detectAutoFace() const
 {
     if (!model) {
         return KPageView::Plain;
     }
 
     /**
      * Check whether the model has sub pages.
      */
     bool hasSubPages = false;
     const int count = model->rowCount();
     for (int i = 0; i < count; ++i) {
         if (model->rowCount(model->index(i, 0)) > 0) {
             hasSubPages = true;
             break;
         }
     }
 
     if (hasSubPages) {
         return KPageView::Tree;
     }
 
     if (model->rowCount() > 1) {
         return KPageView::List;
     }
 
     return KPageView::Plain;
 }
 
 void KPageViewPrivate::_k_modelChanged()
 {
     if (!model) {
         return;
     }
 
     /**
      * If the face type is Auto, we rebuild the GUI whenever the layout
      * of the model changes.
      */
     if (faceType == KPageView::Auto) {
         _k_rebuildGui();
         // If you discover some crashes use the line below instead...
         //QTimer::singleShot(0, q, SLOT(_k_rebuildGui()));
     }
 
     /**
      * Set the stack to the minimum size of the largest widget.
      */
     QSize size = stack->size();
     const QList<QWidget *> widgets = collectPages();
     for (int i = 0; i < widgets.count(); ++i) {
         const QWidget *widget = widgets[ i ];
         if (widget) {
             size = size.expandedTo(widget->minimumSizeHint());
         }
     }
     stack->setMinimumSize(size);
 
     updateSelection();
 }
 
 void KPageViewPrivate::_k_pageSelected(const QItemSelection &index, const QItemSelection &previous)
 {
     if (!model) {
         return;
     }
 
     // Return if the current Index is not valid
     if (index.indexes().size() != 1) {
         return;
     }
     QModelIndex currentIndex = index.indexes().first();
 
     QModelIndex previousIndex;
     // The previous index can be invalid
     if (previous.indexes().size() == 1) {
         previousIndex = previous.indexes().first();
     }
 
     if (faceType != KPageView::Tabbed) {
         QWidget *widget = qvariant_cast<QWidget *>(model->data(currentIndex, KPageModel::WidgetRole));
 
         if (widget) {
             if (stack->indexOf(widget) == -1) {     // not included yet
                 stack->addWidget(widget);
             }
 
             stack->setCurrentWidget(widget);
         } else {
             stack->setCurrentWidget(defaultWidget);
         }
 
         updateTitleWidget(currentIndex);
     }
 
     Q_Q(KPageView);
     emit q->currentPageChanged(currentIndex, previousIndex);
 }
 
 void KPageViewPrivate::updateTitleWidget(const QModelIndex &index)
 {
     Q_Q(KPageView);
 
     const bool headerVisible = model->data(index, KPageModel::HeaderVisibleRole).toBool();
     if (!headerVisible) {
         titleWidget->setVisible(false);
         return;
     }
     QString header = model->data(index, KPageModel::HeaderRole).toString();
     if (header.isNull()) {   //TODO KDE5 remove that ugly logic, see also doxy-comments in KPageWidgetItem::setHeader()
         header = model->data(index, Qt::DisplayRole).toString();
     }
 
     titleWidget->setText(header);
 
     titleWidget->setVisible(q->showPageHeader());
 }
 
 void KPageViewPrivate::_k_dataChanged(const QModelIndex &, const QModelIndex &)
 {
     /**
      * When data has changed we update the header and icon for the currently selected
      * page.
      */
     if (!view) {
         return;
     }
 
     QModelIndex index = view->selectionModel()->currentIndex();
     if (!index.isValid()) {
         return;
     }
 
     updateTitleWidget(index);
 }
 
 KPageViewPrivate::KPageViewPrivate(KPageView *_parent)
     : q_ptr(_parent), model(nullptr), faceType(KPageView::Auto),
       layout(nullptr), stack(nullptr), titleWidget(nullptr), view(nullptr)
 {
 }
 
 void KPageViewPrivate::init()
 {
     Q_Q(KPageView);
     layout = new QGridLayout(q);
     layout->setContentsMargins(0, 0, 0, 0);
     stack = new KPageStackedWidget(q);
     titleWidget = new KTitleWidget(q);
     QPixmap emptyPixmap(22, 22);
     emptyPixmap.fill(Qt::transparent);
     titleWidget->setPixmap(emptyPixmap);
     layout->addWidget(titleWidget, 1, 1);
     layout->addWidget(stack, 2, 1);
 
     defaultWidget = new QWidget(q);
     stack->addWidget(defaultWidget);
 
     // stack should use most space
     layout->setColumnStretch(1, 1);
     layout->setRowStretch(2, 1);
 }
 
 /**
  * KPageView Implementation
  */
 KPageView::KPageView(QWidget *parent)
     : QWidget(parent), d_ptr(new KPageViewPrivate(this))
 {
     d_ptr->init();
 }
 
 KPageView::KPageView(KPageViewPrivate &dd, QWidget *parent)
     : QWidget(parent), d_ptr(&dd)
 {
     d_ptr->init();
 }
 
 KPageView::~KPageView()
 {
     delete d_ptr;
 }
 
 void KPageView::setModel(QAbstractItemModel *model)
 {
     Q_D(KPageView);
     // clean up old model
     if (d->model) {
         disconnect(d->model, SIGNAL(layoutChanged()), this, SLOT(_k_modelChanged()));
         disconnect(d->model, SIGNAL(dataChanged(QModelIndex,QModelIndex)),
                    this, SLOT(_k_dataChanged(QModelIndex,QModelIndex)));
     }
 
     d->model = model;
 
     if (d->model) {
         connect(d->model, SIGNAL(layoutChanged()), this, SLOT(_k_modelChanged()));
         connect(d->model, SIGNAL(dataChanged(QModelIndex,QModelIndex)),
                 this, SLOT(_k_dataChanged(QModelIndex,QModelIndex)));
 
         // set new model in navigation view
         if (d->view) {
             d->view->setModel(model);
         }
     }
 
     d->_k_rebuildGui();
 }
 
 QAbstractItemModel *KPageView::model() const
 {
     return d_func()->model;
 }
 
 void KPageView::setFaceType(FaceType faceType)
 {
     Q_D(KPageView);
     d->faceType = faceType;
 
     d->_k_rebuildGui();
 }
 
 KPageView::FaceType KPageView::faceType() const
 {
     return d_func()->faceType;
 }
 
 void KPageView::setCurrentPage(const QModelIndex &index)
 {
     Q_D(KPageView);
     if (!d->view || !d->view->selectionModel()) {
         return;
     }
 
     d->view->selectionModel()->setCurrentIndex(index, QItemSelectionModel::SelectCurrent);
 }
 
 QModelIndex KPageView::currentPage() const
 {
     Q_D(const KPageView);
     if (!d->view || !d->view->selectionModel()) {
         return QModelIndex();
     }
 
     return d->view->selectionModel()->currentIndex();
 }
 
 void KPageView::setItemDelegate(QAbstractItemDelegate *delegate)
 {
     Q_D(KPageView);
     if (d->view) {
         d->view->setItemDelegate(delegate);
     }
 }
 
 QAbstractItemDelegate *KPageView::itemDelegate() const
 {
     Q_D(const KPageView);
     if (d->view) {
         return d->view->itemDelegate();
     } else {
         return nullptr;
     }
 }
 
 void KPageView::setDefaultWidget(QWidget *widget)
 {
     Q_D(KPageView);
 
     Q_ASSERT(widget);
 
     bool isCurrent = (d->stack->currentIndex() == d->stack->indexOf(d->defaultWidget));
 
     // remove old default widget
     d->stack->removeWidget(d->defaultWidget);
     delete d->defaultWidget;
 
     // add new default widget
     d->defaultWidget = widget;
     d->stack->addWidget(d->defaultWidget);
 
     if (isCurrent) {
         d->stack->setCurrentWidget(d->defaultWidget);
     }
 }
 
 void KPageView::setPageHeader(QWidget *header)
 {
     Q_D(KPageView);
     if (d->pageHeader == header) {
         return;
     }
 
-    if (d->pageHeader == header) {
-        return;
-    }
-
     if (d->pageHeader) {
         d->layout->removeWidget(d->pageHeader);
     }
     d->layout->removeWidget(d->titleWidget);
 
     d->pageHeader = header;
 
     if (d->pageHeader) {
         d->layout->addWidget(d->pageHeader, 1, 1);
         d->pageHeader->setVisible(showPageHeader());
     } else {
         d->layout->addWidget(d->titleWidget, 1, 1);
         d->titleWidget->setVisible(showPageHeader());
     }
-    
 }
 
 QWidget *KPageView::pageHeader() const
 {
     Q_D(const KPageView);
     return d->pageHeader;
 }
 
 void KPageView::setPageFooter(QWidget *footer)
 {
     Q_D(KPageView);
     if (d->pageFooter == footer) {
         return;
     }
 
     if (d->pageFooter) {
         d->layout->removeWidget(d->pageFooter);
     }
 
     d->pageFooter = footer;
 
     if (footer) {
         d->layout->addWidget(d->pageFooter, 3, 1);
     }
 }
 
 QWidget *KPageView::pageFooter() const
 {
     Q_D(const KPageView);
     return d->pageFooter;
 }
 
 QAbstractItemView *KPageView::createView()
 {
     Q_D(KPageView);
     if (d->faceType == Auto) {
         const FaceType faceType = d->detectAutoFace();
 
         if (faceType == Plain) {
             return new KDEPrivate::KPagePlainView(this);
         } else if (faceType == List) {
             return new KDEPrivate::KPageListView(this);
         } else if (faceType == Tree) {
             return new KDEPrivate::KPageTreeView(this);
         } else { // should never happen
             return nullptr;
         }
     } else if (d->faceType == Plain) {
         return new KDEPrivate::KPagePlainView(this);
     } else if (d->faceType == List) {
         return new KDEPrivate::KPageListView(this);
     } else if (d->faceType == Tree) {
         return new KDEPrivate::KPageTreeView(this);
     } else if (d->faceType == Tabbed) {
         return new KDEPrivate::KPageTabbedView(this);
     } else {
         return nullptr;
     }
 }
 
 bool KPageView::showPageHeader() const
 {
     Q_D(const KPageView);
     FaceType faceType = d->faceType;
 
     if (faceType == Auto) {
         faceType = d->detectAutoFace();
     }
 
     if (faceType == Tabbed) {
         return false;
     } else {
         return d->pageHeader || !d->titleWidget->text().isEmpty();
     }
 }
 
 Qt::Alignment KPageView::viewPosition() const
 {
     Q_D(const KPageView);
     FaceType faceType = d->faceType;
 
     if (faceType == Auto) {
         faceType = d->detectAutoFace();
     }
 
     if (faceType == Plain || faceType == Tabbed) {
         return Qt::AlignTop;
     } else {
         return Qt::AlignLeft;
     }
 }
 
 #include "moc_kpageview.cpp"
diff --git a/src/kpageview_p.h b/src/kpageview_p.h
index daad025..a079289 100644
--- a/src/kpageview_p.h
+++ b/src/kpageview_p.h
@@ -1,250 +1,250 @@
 /*
     This file is part of the KDE Libraries
 
     Copyright (C) 2006 Tobias Koenig (tokoe@kde.org)
     Copyright (C) 2007 Rafael Fernández López (ereslibre@kde.org)
 
     This library is free software; you can redistribute it and/or
     modify it under the terms of the GNU Library General Public
     License as published by the Free Software Foundation; either
     version 2 of the License, or (at your option) any later version.
 
     This library is distributed in the hope that it will be useful,
     but WITHOUT ANY WARRANTY; without even the implied warranty of
     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
     Library General Public License for more details.
 
     You should have received a copy of the GNU Library General Public License
     along with this library; see the file COPYING.LIB. If not, write to
     the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
     Boston, MA 02110-1301, USA.
 */
 
 #ifndef KPAGEVIEW_P_H
 #define KPAGEVIEW_P_H
 
 #include "kpageview.h"
 
 #include <QAbstractItemDelegate>
 #include <QGridLayout>
 #include <QStackedWidget>
 #include <QAbstractProxyModel>
 #include <QListView>
 #include <QTabBar>
 #include <QTreeView>
 #include <ktitlewidget.h>
 #include <QPointer>
 
 class KPageStackedWidget : public QStackedWidget
 {
     Q_OBJECT
 public:
     explicit KPageStackedWidget(QWidget *parent = nullptr)
         : QStackedWidget(parent)
     {
     }
 
     void setMinimumSize(const QSize &size)
     {
         mMinimumSize = size;
     }
 
     QSize minimumSizeHint() const override
     {
         return mMinimumSize.expandedTo(QStackedWidget::minimumSizeHint());
     }
 
 private:
     QSize mMinimumSize;
 };
 
 class KPageViewPrivate
 {
     Q_DECLARE_PUBLIC(KPageView)
 protected:
     KPageViewPrivate(KPageView *);
 
     KPageView *q_ptr;
 
     // data
     QAbstractItemModel *model;
     KPageView::FaceType faceType;
 
     // gui
     QGridLayout *layout;
     KPageStackedWidget *stack;
     KTitleWidget *titleWidget;
     QWidget *defaultWidget;
 
     QAbstractItemView *view;
 
-    QPointer <QWidget> pageHeader;
-    QPointer <QWidget> pageFooter;
+    QPointer<QWidget> pageHeader;
+    QPointer<QWidget> pageFooter;
 
     void updateTitleWidget(const QModelIndex &index);
 
     void updateSelection();
     void cleanupPages();
     QList<QWidget *> collectPages(const QModelIndex &parent = QModelIndex());
     KPageView::FaceType detectAutoFace() const;
 
     // private slots
     void _k_rebuildGui();
     void _k_modelChanged();
     void _k_dataChanged(const QModelIndex &, const QModelIndex &);
     void _k_pageSelected(const QItemSelection &, const QItemSelection &);
 
 private:
     void init();
 };
 
 namespace KDEPrivate
 {
 
 class KPageListViewDelegate;
 class KPageListViewProxy;
 
 class KPagePlainView : public QAbstractItemView
 {
     Q_OBJECT
 public:
     explicit KPagePlainView(QWidget *parent = nullptr);
 
     QModelIndex indexAt(const QPoint &point) const override;
     void scrollTo(const QModelIndex &index, ScrollHint hint = EnsureVisible) override;
     QRect visualRect(const QModelIndex &index) const override;
 
 protected:
     QModelIndex moveCursor(QAbstractItemView::CursorAction, Qt::KeyboardModifiers) override;
     int horizontalOffset() const override;
     int verticalOffset() const override;
     bool isIndexHidden(const QModelIndex &) const override;
     void setSelection(const QRect &, QFlags<QItemSelectionModel::SelectionFlag>) override;
     QRegion visualRegionForSelection(const QItemSelection &) const override;
 };
 
 class KPageListView : public QListView
 {
     Q_OBJECT
 
 public:
     explicit KPageListView(QWidget *parent = nullptr);
     ~KPageListView() override;
 
     void setModel(QAbstractItemModel *model) override;
 
 protected:
     void changeEvent(QEvent *event) override;
 
 private Q_SLOTS:
     void updateWidth();
 };
 
 class KPageTreeView : public QTreeView
 {
     Q_OBJECT
 
 public:
     explicit KPageTreeView(QWidget *parent = nullptr);
 
     void setModel(QAbstractItemModel *model) override;
 
 private Q_SLOTS:
     void updateWidth();
 
 private:
     void expandItems(const QModelIndex &index = QModelIndex());
 };
 
 class KPageTabbedView : public QAbstractItemView
 {
     Q_OBJECT
 
 public:
     explicit KPageTabbedView(QWidget *parent = nullptr);
     ~KPageTabbedView() override;
 
     void setModel(QAbstractItemModel *model) override;
 
     QModelIndex indexAt(const QPoint &point) const override;
     void scrollTo(const QModelIndex &index, ScrollHint hint = EnsureVisible) override;
     QRect visualRect(const QModelIndex &index) const override;
 
     QSize minimumSizeHint() const override;
 
 protected:
     QModelIndex moveCursor(QAbstractItemView::CursorAction, Qt::KeyboardModifiers) override;
     int horizontalOffset() const override;
     int verticalOffset() const override;
     bool isIndexHidden(const QModelIndex &) const override;
     void setSelection(const QRect &, QFlags<QItemSelectionModel::SelectionFlag>) override;
     QRegion visualRegionForSelection(const QItemSelection &) const override;
 
 private Q_SLOTS:
     void currentPageChanged(int);
     void layoutChanged();
     void dataChanged(const QModelIndex &, const QModelIndex &, const QVector<int> &roles) override;
 
 private:
     QTabWidget *mTabWidget;
 };
 
 class KPageListViewDelegate : public QAbstractItemDelegate
 {
     Q_OBJECT
 
 public:
     explicit KPageListViewDelegate(QObject *parent = nullptr);
 
     void paint(QPainter *painter, const QStyleOptionViewItem &option, const QModelIndex &index) const override;
     QSize sizeHint(const QStyleOptionViewItem &option, const QModelIndex &index) const override;
 
 private:
     void drawFocus(QPainter *, const QStyleOptionViewItem &, const QRect &) const;
 };
 
 /**
  * We need this proxy model to map the leaves of a tree-like model
  * to a one-level list model.
  */
 class KPageListViewProxy : public QAbstractProxyModel
 {
     Q_OBJECT
 
 public:
     explicit KPageListViewProxy(QObject *parent = nullptr);
     ~KPageListViewProxy() override;
 
     int rowCount(const QModelIndex &parent = QModelIndex()) const override;
     int columnCount(const QModelIndex &parent = QModelIndex()) const override;
     QModelIndex index(int row, int column, const QModelIndex &parent = QModelIndex()) const override;
     QModelIndex parent(const QModelIndex &) const override;
     QVariant data(const QModelIndex &index, int role) const override;
     QModelIndex mapFromSource(const QModelIndex &index) const override;
     QModelIndex mapToSource(const QModelIndex &index) const override;
 
 public Q_SLOTS:
     void rebuildMap();
 
 private:
     void addMapEntry(const QModelIndex &);
 
     QList<QModelIndex> mList;
 };
 
 class SelectionModel : public QItemSelectionModel
 {
     Q_OBJECT
 
 public:
     SelectionModel(QAbstractItemModel *model, QObject *parent);
 
 public Q_SLOTS:
     void clear() override;
     void select(const QModelIndex &index, QItemSelectionModel::SelectionFlags command) override;
     void select(const QItemSelection &selection, QItemSelectionModel::SelectionFlags command) override;
 };
 
 }
 
 #endif