1berry/qtbase
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Implement private visualRect() in QTreeView and QAbstractItemView

Browse Source 
QTreeView::visualRect() returns a given model index's visual rectangle.
The method is used to toggle the background during hovering.
The previous implementation included the row indicator, when the first
row section was hovered. When it was unhovered, the row indicator
remained highlighted, until the mouse had left the view port.

The reason is, that the highlighting implementation changed the
rectangle returned for the first section, to include the row indicator.
The implementation for neutralising a highlighted section relies on
QAbstractItemViewPrivate::setHoverIndex() and
QAbstractItemView::update(). These methods don't know about the row
indicator to be included, and therefore do not update() its rectangle.
As a consequence, the correct background gets painted but not updated
on the screen.

This patch moves the calculation of the visual rectangle to a new
QTreeViewPrivate::visualRect_impl(). In addition to the model index,
the new method expects an enum argument, representing the calculation
rule:
- SingleSection: Calculate the rectangle of the given section.
- FullRow: Returns the rectangle of the entire row, regardless of the
index's column.
- AddRowIndiCatorToFirstCulumn: Adds the row indicator to the rect,
if the model index points to the first column.

The patch updates all calls within QTreeView, to use the private method
with the right calculation rule for the use case at hand. It elminates
manual (and repeated) modifications of the return value.

The patch implements QAbstractItemViewPrivate::visualRect(), which
returns QAbstractItemView::visualRect(). It is overridden in
QTreeViewPrivate, so that QAbstractItemViewPrivate::setHoverIndex()
and QAbstractItemView receive the rectangle including row indicator.

As a drive-by, several local variables have been constified and/or
renamed to indicative variable names.

Fixes: QTBUG-115149
Change-Id: I4838bcf744f87d8cfb259c5d8758fb65e091e9fe
Reviewed-by: Tor Arne Vestbø <tor.arne.vestbo@qt.io>
(cherry picked from commit d35e8ad754ebb08430669cef64f10d34e4277d1f)
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
This commit is contained in:
Axel Spoerl 2023-07-27 14:03:18 +02:00 committed by Qt Cherry-pick Bot
parent a835abd6e3
commit 2a748c164e
4 changed files with 83 additions and 45 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/widgets/itemviews/qabstractitemview.cpp
View File

@ -129,8 +129,8 @@ void QAbstractItemViewPrivate::setHoverIndex(const QPersistentModelIndex &index)
q->update(hover); //update the old one
q->update(index); //update the new one
} else {
QRect oldHoverRect = q->visualRect(hover);
QRect newHoverRect = q->visualRect(index);
const QRect oldHoverRect = visualRect(hover);
const QRect newHoverRect = visualRect(index);
viewport->update(QRect(0, newHoverRect.y(), viewport->width(), newHoverRect.height()));
viewport->update(QRect(0, oldHoverRect.y(), viewport->width(), oldHoverRect.height()));
}
@ -3354,7 +3354,7 @@ void QAbstractItemView::update(const QModelIndex &index)
{
Q_D(QAbstractItemView);
if (index.isValid()) {
const QRect rect = visualRect(index);
const QRect rect = d->visualRect(index);
//this test is important for performance reason
//For example in dataChanged we simply update all the cells without checking
//it can be a major bottleneck to update rects that aren't even part of the viewport

2
src/widgets/itemviews/qabstractitemview_p.h
View File

@ -415,6 +415,8 @@ public:
bool verticalScrollModeSet;
bool horizontalScrollModeSet;
virtual QRect visualRect(const QModelIndex &index) const { return q_func()->visualRect(index); }
private:
inline QAbstractItemDelegate *delegateForIndex(const QModelIndex &index) const {
QMap<int, QPointer<QAbstractItemDelegate> >::ConstIterator it;

105
src/widgets/itemviews/qtreeview.cpp
View File

@ -1067,33 +1067,64 @@ void QTreeView::keyboardSearch(const QString &search)
QRect QTreeView::visualRect(const QModelIndex &index) const
{
Q_D(const QTreeView);
return d->visualRect(index, QTreeViewPrivate::SingleSection);
}
if (!d->isIndexValid(index) || isIndexHidden(index))
/*!
\internal
\return the visual rectangle at \param index, according to \param rule.
\list
\li SingleSection
The return value matches the section, which \a index points to.
\li FullRow
Return the rectangle of the entire row, no matter which section
\a index points to.
\li AddRowIndicatorToFirstSection
Like SingleSection. If \index points to the first section, add the
row indicator and its margins.
\endlist
*/
QRect QTreeViewPrivate::visualRect(const QModelIndex &index, RectRule rule) const
{
Q_Q(const QTreeView);
if (!isIndexValid(index))
return QRect();
d->executePostedLayout();
int vi = d->viewIndex(index);
if (vi < 0)
// Calculate the entire row's rectangle, even if one of the elements is hidden
if (q->isIndexHidden(index) && rule != FullRow)
return QRect();
bool spanning = d->viewItems.at(vi).spanning;
executePostedLayout();
const int viewIndex = this->viewIndex(index);
if (viewIndex < 0)
return QRect();
const bool spanning = viewItems.at(viewIndex).spanning;
const int column = index.column();
// if we have a spanning item, make the selection stretch from left to right
int x = (spanning ? 0 : columnViewportPosition(index.column()));
int w = (spanning ? d->header->length() : columnWidth(index.column()));
// handle indentation
if (d->isTreePosition(index.column())) {
int i = d->indentationForItem(vi);
w -= i;
if (!isRightToLeft())
x += i;
int x = (spanning ? 0 : q->columnViewportPosition(column));
int width = (spanning ? header->length() : q->columnWidth(column));
const bool addIndentation = isTreePosition(column) && (column > 0 || rule == SingleSection);
if (rule == FullRow) {
x = 0;
width = q->viewport()->width();
} else if (addIndentation) {
// calculate indentation
const int indentation = indentationForItem(viewIndex);
width -= indentation;
if (!q->isRightToLeft())
x += indentation;
}
int y = d->coordinateForItem(vi);
int h = d->itemHeight(vi);
const int y = coordinateForItem(viewIndex);
const int height = itemHeight(viewIndex);
return QRect(x, y, w, h);
return QRect(x, y, width, height);
}
/*!
@ -1276,16 +1307,13 @@ bool QTreeView::viewportEvent(QEvent *event)
case QEvent::HoverLeave:
case QEvent::HoverMove: {
QHoverEvent *he = static_cast<QHoverEvent*>(event);
int oldBranch = d->hoverBranch;
const int oldBranch = d->hoverBranch;
d->hoverBranch = d->itemDecorationAt(he->position().toPoint());
QModelIndex newIndex = indexAt(he->position().toPoint());
if (d->hover != newIndex || d->hoverBranch != oldBranch) {
// Update the whole hovered over row. No need to update the old hovered
// row, that is taken care in superclass hover handling.
QRect rect = visualRect(newIndex);
rect.setX(0);
rect.setWidth(viewport()->width());
viewport()->update(rect);
viewport()->update(d->visualRect(newIndex, QTreeViewPrivate::FullRow));
}
break; }
default:
@ -1378,8 +1406,6 @@ void QTreeViewPrivate::_q_modelDestroyed()
QRect QTreeViewPrivate::intersectedRect(const QRect rect, const QModelIndex &topLeft, const QModelIndex &bottomRight) const
{
Q_Q(const QTreeView);
const auto parentIdx = topLeft.parent();
executePostedLayout();
QRect updateRect;
@ -1388,7 +1414,7 @@ QRect QTreeViewPrivate::intersectedRect(const QRect rect, const QModelIndex &top
continue;
for (int c = topLeft.column(); c <= bottomRight.column(); ++c) {
const QModelIndex idx(model->index(r, c, parentIdx));
updateRect |= q->visualRect(idx);
updateRect |= visualRect(idx, SingleSection);
}
}
return rect.intersected(updateRect);
@ -1495,8 +1521,9 @@ void QTreeView::drawTree(QPainter *painter, const QRegion &region) const
// paint the visible rows
for (; i < viewItems.size() && y <= area.bottom(); ++i) {
const QModelIndex &index = viewItems.at(i).index;
const int itemHeight = d->itemHeight(i);
option.rect.setRect(0, y, viewportWidth, itemHeight);
option.rect = d->visualRect(index, QTreeViewPrivate::FullRow);
option.state = state | (viewItems.at(i).expanded ? QStyle::State_Open : QStyle::State_None)
| (viewItems.at(i).hasChildren ? QStyle::State_Children : QStyle::State_None)
| (viewItems.at(i).hasMoreSiblings ? QStyle::State_Sibling : QStyle::State_None);
@ -2366,7 +2393,7 @@ QRegion QTreeView::visualRegionForSelection(const QItemSelection &selection) con
}
if (!leftIndex.isValid())
continue;
const QRect leftRect = visualRect(leftIndex);
const QRect leftRect = d->visualRect(leftIndex, QTreeViewPrivate::SingleSection);
int top = leftRect.top();
QModelIndex rightIndex = range.bottomRight();
while (rightIndex.isValid() && isIndexHidden(rightIndex)) {
@ -2377,7 +2404,7 @@ QRegion QTreeView::visualRegionForSelection(const QItemSelection &selection) con
}
if (!rightIndex.isValid())
continue;
const QRect rightRect = visualRect(rightIndex);
const QRect rightRect = d->visualRect(rightIndex, QTreeViewPrivate::SingleSection);
int bottom = rightRect.bottom();
if (top > bottom)
qSwap<int>(top, bottom);
@ -2644,7 +2671,8 @@ QSize QTreeView::viewportSizeHint() const
return QAbstractItemView::viewportSizeHint();
// Get rect for last item
const QRect deepestRect = visualRect(d->viewItems.last().index);
const QRect deepestRect = d->visualRect(d->viewItems.last().index,
QTreeViewPrivate::SingleSection);
if (!deepestRect.isValid())
return QAbstractItemView::viewportSizeHint();
@ -3256,7 +3284,7 @@ QPixmap QTreeViewPrivate::renderTreeToPixmapForAnimation(const QRect &rect) cons
for (QEditorIndexHash::const_iterator it = editorIndexHash.constBegin(); it != editorIndexHash.constEnd(); ++it) {
QWidget *editor = it.key();
const QModelIndex &index = it.value();
option.rect = q->visualRect(index);
option.rect = visualRect(index, SingleSection);
if (option.rect.isValid()) {
if (QAbstractItemDelegate *delegate = q->itemDelegateForIndex(index))
@ -3998,21 +4026,14 @@ void QTreeViewPrivate::updateIndentationFromStyle()
*/
void QTreeView::currentChanged(const QModelIndex &current, const QModelIndex &previous)
{
Q_D(QTreeView);
QAbstractItemView::currentChanged(current, previous);
if (allColumnsShowFocus()) {
if (previous.isValid()) {
QRect previousRect = visualRect(previous);
previousRect.setX(0);
previousRect.setWidth(viewport()->width());
viewport()->update(previousRect);
}
if (current.isValid()) {
QRect currentRect = visualRect(current);
currentRect.setX(0);
currentRect.setWidth(viewport()->width());
viewport()->update(currentRect);
}
if (previous.isValid())
viewport()->update(d->visualRect(previous, QTreeViewPrivate::FullRow));
if (current.isValid())
viewport()->update(d->visualRect(current, QTreeViewPrivate::FullRow));
}
#if QT_CONFIG(accessibility)
if (QAccessible::isActive() && current.isValid()) {

15
src/widgets/itemviews/qtreeview_p.h
View File

@ -150,6 +150,21 @@ public:
QList<QStyleOptionViewItem::ViewItemPosition> *itemPositions, int left,
int right) const;
int widthHintForIndex(const QModelIndex &index, int hint, const QStyleOptionViewItem &option, int i) const;
enum RectRule {
FullRow,
SingleSection,
AddRowIndicatorToFirstSection
};
// Base class will get the first visual rect including row indicator
QRect visualRect(const QModelIndex &index) const override
{
return visualRect(index, AddRowIndicatorToFirstSection);
}
QRect visualRect(const QModelIndex &index, RectRule rule) const;
QHeaderView *header;
int indent;