From d3faa3f9ae7e7dc944768e5e77755d14be44fa1e Mon Sep 17 00:00:00 2001 From: Akseli Lahtinen Date: Wed, 9 Apr 2025 07:57:20 +0000 Subject: [PATCH] [PATCH] Show single-click selection emblem when using single-click mouse mode In open/save dialog, we had no way to select items when using single-click mouse mode. (Except holding down control key). This adds the same emblem that Dolphin uses for its selections. It is shown in both treeviews and listviews, but only when single-click mode is active and the filewidget allows selecting multiple items. BUG: 185793 FIXED-IN: 6.14 Gbp-Pq: Name upstream_d8441b7b_Show-single-click-selection-emblem-when-using-single-click-mouse-mode.patch --- src/filewidgets/CMakeLists.txt | 1 + src/filewidgets/kdiroperator.cpp | 13 ++-- src/filewidgets/kdiroperatordetailview.cpp | 31 +++++++++- src/filewidgets/kdiroperatordetailview_p.h | 7 ++- src/filewidgets/kdiroperatoriconview.cpp | 33 +++++++++- src/filewidgets/kdiroperatoriconview_p.h | 9 ++- src/filewidgets/kfileitemselectionemblem.cpp | 63 ++++++++++++++++++++ src/filewidgets/kfileitemselectionemblem.h | 39 ++++++++++++ src/widgets/kfileitemdelegate.cpp | 58 ++++++++++++++++++ src/widgets/kfileitemdelegate.h | 15 +++++ 10 files changed, 259 insertions(+), 10 deletions(-) create mode 100644 src/filewidgets/kfileitemselectionemblem.cpp create mode 100644 src/filewidgets/kfileitemselectionemblem.h diff --git a/src/filewidgets/CMakeLists.txt b/src/filewidgets/CMakeLists.txt index e5fc87f..cf45201 100644 --- a/src/filewidgets/CMakeLists.txt +++ b/src/filewidgets/CMakeLists.txt @@ -22,6 +22,7 @@ target_sources(KF6KIOFileWidgets PRIVATE kdiroperatoriconview.cpp kdirsortfilterproxymodel.cpp #used in combination with kdirmodel.cpp kencodingfiledialog.cpp + kfileitemselectionemblem.cpp kfilebookmarkhandler.cpp kfilecopytomenu.cpp kfilecustomdialog.cpp diff --git a/src/filewidgets/kdiroperator.cpp b/src/filewidgets/kdiroperator.cpp index d2d8a55..e1d3afe 100644 --- a/src/filewidgets/kdiroperator.cpp +++ b/src/filewidgets/kdiroperator.cpp @@ -17,6 +17,7 @@ #include "kdiroperatoriconview_p.h" #include "kdirsortfilterproxymodel.h" #include "kfileitem.h" +#include "kfileitemselectionemblem.h" #include "kfilemetapreview_p.h" #include "knewfilemenu.h" #include "kpreviewwidgetbase.h" @@ -1260,9 +1261,13 @@ bool KDirOperator::eventFilter(QObject *watched, QEvent *event) if (d->m_isTouchEvent) { return true; } - if (d->m_preview && !d->m_preview->isHidden()) { - const QModelIndex hoveredIndex = d->m_itemView->indexAt(d->m_itemView->viewport()->mapFromGlobal(QCursor::pos())); + const QModelIndex hoveredIndex = d->m_itemView->indexAt(d->m_itemView->viewport()->mapFromGlobal(QCursor::pos())); + if (hoveredIndex.isValid()) { + KFileItemSelectionEmblem(d->m_itemView, hoveredIndex, this).updateSelectionEmblemRectForIndex(iconSize()); + } + + if (d->m_preview && !d->m_preview->isHidden()) { if (d->m_lastHoveredIndex == hoveredIndex) { return QWidget::eventFilter(watched, event); } @@ -1510,11 +1515,11 @@ QAbstractItemView *KDirOperator::createView(QWidget *parent, KFile::FileView vie { QAbstractItemView *itemView = nullptr; if (KFile::isDetailView(viewKind) || KFile::isTreeView(viewKind) || KFile::isDetailTreeView(viewKind)) { - KDirOperatorDetailView *detailView = new KDirOperatorDetailView(parent); + KDirOperatorDetailView *detailView = new KDirOperatorDetailView(this, parent); detailView->setViewMode(viewKind); itemView = detailView; } else { - itemView = new KDirOperatorIconView(parent, decorationPosition()); + itemView = new KDirOperatorIconView(this, parent, decorationPosition()); } return itemView; diff --git a/src/filewidgets/kdiroperatordetailview.cpp b/src/filewidgets/kdiroperatordetailview.cpp index 8bbd3e4..b693e6d 100644 --- a/src/filewidgets/kdiroperatordetailview.cpp +++ b/src/filewidgets/kdiroperatordetailview.cpp @@ -5,6 +5,7 @@ */ #include "kdiroperatordetailview_p.h" +#include "kfileitemselectionemblem.h" #include #include @@ -17,9 +18,11 @@ #include #include -KDirOperatorDetailView::KDirOperatorDetailView(QWidget *parent) +KDirOperatorDetailView::KDirOperatorDetailView(KDirOperator *dirOperator, QWidget *parent) : QTreeView(parent) , m_hideDetailColumns(false) + , m_isEmblemClicked(false) + , m_dirOperator(dirOperator) { setRootIsDecorated(false); setSortingEnabled(true); @@ -115,9 +118,15 @@ void KDirOperatorDetailView::dragEnterEvent(QDragEnterEvent *event) void KDirOperatorDetailView::mousePressEvent(QMouseEvent *event) { + const QModelIndex index = indexAt(event->pos()); + // When selection emblem is clicked, select it and don't do anything else + m_isEmblemClicked = KFileItemSelectionEmblem(this, index, m_dirOperator).handleMousePressEvent(event->pos()); + if (m_isEmblemClicked) { + return; + } + QTreeView::mousePressEvent(event); - const QModelIndex index = indexAt(event->pos()); if (!index.isValid() || (index.column() != KDirModel::Name)) { const Qt::KeyboardModifiers modifiers = QApplication::keyboardModifiers(); if (!(modifiers & Qt::ShiftModifier) && !(modifiers & Qt::ControlModifier)) { @@ -126,6 +135,24 @@ void KDirOperatorDetailView::mousePressEvent(QMouseEvent *event) } } +void KDirOperatorDetailView::mouseMoveEvent(QMouseEvent *event) +{ + // Disallow selection dragging when emblem is clicked + if (m_isEmblemClicked) { + return; + } + QTreeView::mouseMoveEvent(event); +} + +void KDirOperatorDetailView::mouseReleaseEvent(QMouseEvent *event) +{ + // Reset the emblem selection + if (m_isEmblemClicked) { + m_isEmblemClicked = false; + } + QTreeView::mouseReleaseEvent(event); +} + void KDirOperatorDetailView::currentChanged(const QModelIndex ¤t, const QModelIndex &previous) { QTreeView::currentChanged(current, previous); diff --git a/src/filewidgets/kdiroperatordetailview_p.h b/src/filewidgets/kdiroperatordetailview_p.h index f30ab94..df45142 100644 --- a/src/filewidgets/kdiroperatordetailview_p.h +++ b/src/filewidgets/kdiroperatordetailview_p.h @@ -7,6 +7,7 @@ #ifndef KDIROPERATORDETAILVIEW_P_H #define KDIROPERATORDETAILVIEW_P_H +#include #include #include @@ -22,7 +23,7 @@ class KDirOperatorDetailView : public QTreeView Q_OBJECT public: - explicit KDirOperatorDetailView(QWidget *parent = nullptr); + explicit KDirOperatorDetailView(KDirOperator *dirOperator, QWidget *parent = nullptr); ~KDirOperatorDetailView() override; /** @@ -36,10 +37,14 @@ protected: bool event(QEvent *event) override; void dragEnterEvent(QDragEnterEvent *event) override; void mousePressEvent(QMouseEvent *event) override; + void mouseMoveEvent(QMouseEvent *event) override; + void mouseReleaseEvent(QMouseEvent *event) override; void currentChanged(const QModelIndex ¤t, const QModelIndex &previous) override; private: bool m_hideDetailColumns; + bool m_isEmblemClicked; + KDirOperator *m_dirOperator; }; #endif diff --git a/src/filewidgets/kdiroperatoriconview.cpp b/src/filewidgets/kdiroperatoriconview.cpp index ace0c00..9180ea1 100644 --- a/src/filewidgets/kdiroperatoriconview.cpp +++ b/src/filewidgets/kdiroperatoriconview.cpp @@ -6,6 +6,7 @@ */ #include "kdiroperatoriconview_p.h" +#include "kfileitemselectionemblem.h" #include #include @@ -15,8 +16,10 @@ #include #include -KDirOperatorIconView::KDirOperatorIconView(QWidget *parent, QStyleOptionViewItem::Position aDecorationPosition) +KDirOperatorIconView::KDirOperatorIconView(KDirOperator *dirOperator, QWidget *parent, QStyleOptionViewItem::Position aDecorationPosition) : QListView(parent) + , m_isEmblemClicked(false) + , m_dirOperator(dirOperator) { setViewMode(QListView::IconMode); setResizeMode(QListView::Adjust); @@ -74,7 +77,15 @@ void KDirOperatorIconView::dragEnterEvent(QDragEnterEvent *event) void KDirOperatorIconView::mousePressEvent(QMouseEvent *event) { - if (!indexAt(event->pos()).isValid()) { + const QModelIndex index = indexAt(event->pos()); + + // When selection emblem is clicked, select it and don't do anything else + m_isEmblemClicked = KFileItemSelectionEmblem(this, index, m_dirOperator).handleMousePressEvent(event->pos()); + if (m_isEmblemClicked) { + return; + } + + if (!index.isValid()) { const Qt::KeyboardModifiers modifiers = QApplication::keyboardModifiers(); if (!(modifiers & Qt::ShiftModifier) && !(modifiers & Qt::ControlModifier)) { clearSelection(); @@ -84,6 +95,24 @@ void KDirOperatorIconView::mousePressEvent(QMouseEvent *event) QListView::mousePressEvent(event); } +void KDirOperatorIconView::mouseMoveEvent(QMouseEvent *event) +{ + // Disallow selection dragging when emblem is clicked + if (m_isEmblemClicked) { + return; + } + QListView::mouseMoveEvent(event); +} + +void KDirOperatorIconView::mouseReleaseEvent(QMouseEvent *event) +{ + // Reset the emblem selection + if (m_isEmblemClicked) { + m_isEmblemClicked = false; + } + QListView::mouseReleaseEvent(event); +} + void KDirOperatorIconView::wheelEvent(QWheelEvent *event) { QListView::wheelEvent(event); diff --git a/src/filewidgets/kdiroperatoriconview_p.h b/src/filewidgets/kdiroperatoriconview_p.h index c4ab967..f1caba9 100644 --- a/src/filewidgets/kdiroperatoriconview_p.h +++ b/src/filewidgets/kdiroperatoriconview_p.h @@ -8,6 +8,7 @@ #ifndef KDIROPERATORICONVIEW_P_H #define KDIROPERATORICONVIEW_P_H +#include #include /** @@ -18,7 +19,9 @@ class KDirOperatorIconView : public QListView { Q_OBJECT public: - KDirOperatorIconView(QWidget *parent = nullptr, QStyleOptionViewItem::Position decorationPosition = QStyleOptionViewItem::Position::Top); + KDirOperatorIconView(KDirOperator *dirOperator, + QWidget *parent = nullptr, + QStyleOptionViewItem::Position decorationPosition = QStyleOptionViewItem::Position::Top); ~KDirOperatorIconView() override; void setDecorationPosition(QStyleOptionViewItem::Position decorationPosition); @@ -26,6 +29,8 @@ protected: void initViewItemOption(QStyleOptionViewItem *option) const override; void dragEnterEvent(QDragEnterEvent *event) override; void mousePressEvent(QMouseEvent *event) override; + void mouseMoveEvent(QMouseEvent *event) override; + void mouseReleaseEvent(QMouseEvent *event) override; void wheelEvent(QWheelEvent *event) override; void resizeEvent(QResizeEvent *event) override; @@ -34,6 +39,8 @@ protected private: QStyleOptionViewItem::Position decorationPosition; + bool m_isEmblemClicked; + KDirOperator *m_dirOperator; }; #endif // KDIROPERATORICONVIEW_P_H diff --git a/src/filewidgets/kfileitemselectionemblem.cpp b/src/filewidgets/kfileitemselectionemblem.cpp new file mode 100644 index 0000000..566dee3 --- /dev/null +++ b/src/filewidgets/kfileitemselectionemblem.cpp @@ -0,0 +1,63 @@ +/* + SPDX-FileCopyrightText: 2025 Akseli Lahtinen + + SPDX-License-Identifier: LGPL-2.0-or-later +*/ + +#include "kfileitemselectionemblem.h" +#include "kfileitemdelegate.h" + +#include +#include +#include +#include +#include + +KFileItemSelectionEmblem::KFileItemSelectionEmblem(QAbstractItemView *itemView, QModelIndex index, KDirOperator *dirOperator) +{ + m_itemView = itemView; + m_index = index; + m_fileItemDelegate = fileItemDelegate(); + m_dirOperator = dirOperator; + m_fileItem = m_fileItemDelegate->fileItem(m_index); +} + +KFileItemSelectionEmblem::~KFileItemSelectionEmblem() +{ +} + +bool KFileItemSelectionEmblem::isEmblemEnabled() +{ + if (m_itemView->selectionMode() == QAbstractItemView::ExtendedSelection && qApp->style()->styleHint(QStyle::SH_ItemView_ActivateItemOnSingleClick)) { + if (m_fileItem.isDir()) { + return m_dirOperator->isSelected(m_fileItem); + } + return true; + } + return false; +} + +KFileItemDelegate *KFileItemSelectionEmblem::fileItemDelegate() +{ + auto itemDelegate = m_itemView->itemDelegateForIndex(m_index); + if (itemDelegate) { + return qobject_cast(itemDelegate); + } + return nullptr; +} + +void KFileItemSelectionEmblem::updateSelectionEmblemRectForIndex(const int iconSize) +{ + if (isEmblemEnabled() && m_fileItemDelegate) { + m_fileItemDelegate->setSelectionEmblemRect(m_itemView->visualRect(m_index), iconSize); + } +} + +bool KFileItemSelectionEmblem::handleMousePressEvent(const QPoint mousePos) +{ + if (isEmblemEnabled() && m_fileItemDelegate && m_fileItemDelegate->selectionEmblemRect().contains(mousePos)) { + m_itemView->selectionModel()->select(m_index, QItemSelectionModel::Toggle); + return true; + } + return false; +} diff --git a/src/filewidgets/kfileitemselectionemblem.h b/src/filewidgets/kfileitemselectionemblem.h new file mode 100644 index 0000000..4bfd78a --- /dev/null +++ b/src/filewidgets/kfileitemselectionemblem.h @@ -0,0 +1,39 @@ +/* + SPDX-FileCopyrightText: 2025 Akseli Lahtinen + + SPDX-License-Identifier: LGPL-2.0-or-later +*/ + +#ifndef KFILEITEMSELECTIONEMBLEM_H +#define KFILEITEMSELECTIONEMBLEM_H + +#include +#include +#include + +class KFileItem; +class KFileItemDelegate; +class QPoint; + +class KFileItemSelectionEmblem +{ +public: + KFileItemSelectionEmblem(QAbstractItemView *itemView, QModelIndex index, KDirOperator *dirOperator); + ~KFileItemSelectionEmblem(); + + void updateSelectionEmblemRectForIndex(const int iconSize); + bool handleMousePressEvent(const QPoint mousePos); + bool isEmblemEnabled(); + +private: + KFileItemDelegate *fileItemDelegate(); + + QAbstractItemView *m_itemView; + QModelIndex m_index; + KDirOperator *m_dirOperator; + KFileItemDelegate *m_fileItemDelegate; + KFileItem m_fileItem; + bool m_isDir; +}; + +#endif diff --git a/src/widgets/kfileitemdelegate.cpp b/src/widgets/kfileitemdelegate.cpp index c00c695..83acd99 100644 --- a/src/widgets/kfileitemdelegate.cpp +++ b/src/widgets/kfileitemdelegate.cpp @@ -101,6 +101,7 @@ public: void gotNewIcon(const QModelIndex &index); void paintJobTransfers(QPainter *painter, const qreal &jobAnimationAngle, const QPoint &iconPos, const QStyleOptionViewItem &opt); + int scaledEmblemSize(int iconSize) const; public: KFileItemDelegate::InformationList informationList; @@ -112,6 +113,7 @@ public: QTextOption::WrapMode wrapMode; bool jobTransfersVisible; QIcon downArrowIcon; + QRect emblemRect; private: KIO::DelegateAnimationHandler *animationHandler; @@ -128,6 +130,7 @@ KFileItemDelegate::Private::Private(KFileItemDelegate *parent) , showToolTipWhenElided(true) , wrapMode(QTextOption::WrapAtWordBoundaryOrAnywhere) , jobTransfersVisible(false) + , emblemRect(QRect()) , animationHandler(new KIO::DelegateAnimationHandler(parent)) , activeMargins(nullptr) { @@ -1153,6 +1156,7 @@ void KFileItemDelegate::paint(QPainter *painter, const QStyleOptionViewItem &opt QPixmap icon = opt.icon.pixmap(opt.decorationSize, iconMode, iconState); const KFileItem fileItem = d->fileItem(index); + const bool isDir = fileItem.isDir(); if (fileItem.isHidden()) { KIconEffect::semiTransparent(icon); } @@ -1176,6 +1180,7 @@ void KFileItemDelegate::paint(QPainter *painter, const QStyleOptionViewItem &opt pixmap = d->transition(fadeFromPixmap, pixmap, state->fadeProgress()); } painter->drawPixmap(option.rect.topLeft(), pixmap); + drawSelectionEmblem(option, painter, index); if (d->jobTransfersVisible && index.column() == 0) { if (index.data(KDirModel::HasJobRole).toBool()) { d->paintJobTransfers(painter, state->jobAnimationAngle(), iconPos, opt); @@ -1251,6 +1256,7 @@ void KFileItemDelegate::paint(QPainter *painter, const QStyleOptionViewItem &opt p.setRenderHint(QPainter::Antialiasing); style->drawPrimitive(QStyle::PE_PanelItemViewItem, &opt, &p, opt.widget); p.drawPixmap(iconPos, icon); + drawSelectionEmblem(option, painter, index); d->drawTextItems(&p, labelLayout, labelColor, infoLayout, infoColor, textBoundingRect); d->drawFocusRect(&p, opt, focusRect); p.end(); @@ -1263,6 +1269,7 @@ void KFileItemDelegate::paint(QPainter *painter, const QStyleOptionViewItem &opt p.setRenderHint(QPainter::Antialiasing); style->drawPrimitive(QStyle::PE_PanelItemViewItem, &opt, &p, opt.widget); p.drawPixmap(iconPos, icon); + drawSelectionEmblem(option, painter, index); d->drawTextItems(&p, labelLayout, labelColor, infoLayout, infoColor, textBoundingRect); d->drawFocusRect(&p, opt, focusRect); p.end(); @@ -1282,6 +1289,7 @@ void KFileItemDelegate::paint(QPainter *painter, const QStyleOptionViewItem &opt } painter->drawPixmap(option.rect.topLeft(), pixmap); + drawSelectionEmblem(option, painter, index); painter->setRenderHint(QPainter::Antialiasing); if (d->jobTransfersVisible && index.column() == 0) { if (index.data(KDirModel::HasJobRole).toBool()) { @@ -1306,6 +1314,7 @@ void KFileItemDelegate::paint(QPainter *painter, const QStyleOptionViewItem &opt d->drawTextItems(painter, labelLayout, labelColor, infoLayout, infoColor, textBoundingRect); d->drawFocusRect(painter, opt, focusRect); + drawSelectionEmblem(option, painter, index); if (d->jobTransfersVisible && index.column() == 0 && state) { if (index.data(KDirModel::HasJobRole).toBool()) { @@ -1315,6 +1324,33 @@ void KFileItemDelegate::paint(QPainter *painter, const QStyleOptionViewItem &opt painter->restore(); } +void KFileItemDelegate::drawSelectionEmblem(QStyleOptionViewItem option, QPainter *painter, const QModelIndex &index) const +{ + if (index.column() != 0 || !qApp->style()->styleHint(QStyle::SH_ItemView_ActivateItemOnSingleClick)) { + return; + } + const auto state = option.state; + if ((state & QStyle::State_MouseOver && !fileItem(index).isDir()) || (state & QStyle::State_Selected)) { + const QString selectionEmblem = state & QStyle::State_Selected ? QStringLiteral("emblem-remove") : QStringLiteral("emblem-added"); + const auto emblem = QIcon::fromTheme(selectionEmblem).pixmap(d->emblemRect.size(), state & QStyle::State_MouseOver ? QIcon::Active : QIcon::Disabled); + + painter->drawPixmap(d->emblemRect.topLeft(), emblem); + } +} + +int KFileItemDelegate::Private::scaledEmblemSize(int iconSize) const +{ + if (iconSize <= KIconLoader::SizeSmallMedium) { + return KIconLoader::SizeSmall; + } else if (iconSize <= KIconLoader::SizeHuge) { + return KIconLoader::SizeSmallMedium; + } else if (iconSize <= KIconLoader::SizeEnormous) { + return KIconLoader::SizeMedium; + } + + return KIconLoader::SizeHuge; +} + QWidget *KFileItemDelegate::createEditor(QWidget *parent, const QStyleOptionViewItem &option, const QModelIndex &index) const { QStyleOptionViewItem opt(option); @@ -1551,4 +1587,26 @@ bool KFileItemDelegate::eventFilter(QObject *object, QEvent *event) } // switch (event->type()) } +void KFileItemDelegate::setSelectionEmblemRect(QRect rect, int iconSize) +{ + const auto emblemSize = d->scaledEmblemSize(iconSize); + + // With small icons, try to center the emblem on top of the icon + if (iconSize <= KIconLoader::SizeSmallMedium) { + d->emblemRect = QRect(rect.topLeft().x() + emblemSize / 4, rect.topLeft().y() + emblemSize / 4, emblemSize, emblemSize); + } else { + d->emblemRect = QRect(rect.topLeft().x(), rect.topLeft().y(), emblemSize, emblemSize); + } +} + +QRect KFileItemDelegate::selectionEmblemRect() const +{ + return d->emblemRect; +} + +KFileItem KFileItemDelegate::fileItem(const QModelIndex &index) const +{ + return d->fileItem(index); +} + #include "moc_kfileitemdelegate.cpp" diff --git a/src/widgets/kfileitemdelegate.h b/src/widgets/kfileitemdelegate.h index b4dd7eb..83ecfe4 100644 --- a/src/widgets/kfileitemdelegate.h +++ b/src/widgets/kfileitemdelegate.h @@ -9,6 +9,7 @@ #define KFILEITEMDELEGATE_H #include "kiowidgets_export.h" +#include #include #include @@ -398,6 +399,18 @@ public: */ bool eventFilter(QObject *object, QEvent *event) override; + /** + * @return The rectangle where selectionEmblem is being drawn + */ + QRect selectionEmblemRect() const; + + /** + * Set the rectangle where selectionEmblem should be drawn in. + */ + void setSelectionEmblemRect(QRect rect, int iconSize); + + KFileItem fileItem(const QModelIndex &index) const; + public Q_SLOTS: /** * Reimplemented from @ref QAbstractItemDelegate. @@ -414,6 +427,8 @@ private: class Private; std::unique_ptr const d; /// @internal Q_DISABLE_COPY(KFileItemDelegate) + + void drawSelectionEmblem(QStyleOptionViewItem option, QPainter *painter, const QModelIndex &index) const; }; #endif // KFILEITEMDELEGATE_H -- 2.30.2