QItemDelegate + QListView : paint

[Killian]

Hi guys,

I'm trying to polish a small tool I wrote and therefore want to implement an item delegate for my list view.

Scenario:
There's a QStandardItemModel which is assigned to the list view. The view also has a delegate.
I just added to items to the model which are shown without any problem.

Issue:
There's something wrong with the hovering background color. Please see my code below.

painter->setPen(Qt::NoPen);
painter->setBrush(QColor(0, 0, 255));
 
if(option.state & QStyle::State_MouseOver)
{
    painter->setBrush(QColor(255, 0, 0));
}
 
painter->drawRect(option.rect);
 
painter->setPen(fontColor);
painter->drawText(option.rect.left() + 5, option.rect.top() + option.rect.height() / 2 + 4, qvariant_cast<QString>(index.data()));

painter->setPen(Qt::NoPen);
painter->setBrush(QColor(0, 0, 255));

if(option.state & QStyle::State_MouseOver)
{
    painter->setBrush(QColor(255, 0, 0));
}

painter->drawRect(option.rect);

painter->setPen(fontColor);
painter->drawText(option.rect.left() + 5, option.rect.top() + option.rect.height() / 2 + 4, qvariant_cast<QString>(index.data()));

To copy to clipboard, switch view to plain text mode 

The item's text is drawn correctly at the desired position. But the background is drawn over both items.

The sizeHint() return this:

return QSize(option.rect.width(), 20);

return QSize(option.rect.width(), 20);

To copy to clipboard, switch view to plain text mode 

I'm sure I'm missing something or am doing something wrong.
Could anybody please shed some light? :-)

Thanks in advance.

[anda_skoa]

Have you tried saving and restoring the old values for pen and brush, i.e. not permanently change painter state?

Something like

const QPen oldPen = painter->pen(),
const QBrush oldBrush = painter->brush();
 
// your code
 
painter->setPen(oldPen);
painter->setBrush(oldBrush);

const QPen oldPen = painter->pen(),
const QBrush oldBrush = painter->brush();

// your code

painter->setPen(oldPen);
painter->setBrush(oldBrush);

To copy to clipboard, switch view to plain text mode 

Cheers,
_

[Killian]

Well, the issue is not related to wrong painter states but rather to the area where the painter draws the rectangle for the item background.
I assumed that option.rect would deliver the exact boundaries of the hovered item, but if I hover over the first item in the view the second is also highlighted.

Don't know if my explanation makes it clear, please let me know if more details/code is required to help.

Thanks.

[anda_skoa]

Hmm, weird, so option.rect is larger than the cell of that list view entry?

But the text positioning on the same values is correct at the same time?

Cheers,
_

[Killian]

Exactly! That's what makes me think I'm doing something wrong with the background. Just can't see what that would be.

[anda_skoa]

Have you tried not changing the brush but using QPainter::fillRect() instead?

Cheers,
_

[Killian]

I also tried that but still to no avail. So I guess there must be something wrong somewhere else in my delegate class. I've done delegates so many times but maybe I just dont see the wood for the trees.
Anyway, it's just an esthetic thingy so I won't spend any more time on it right now.

Thanks for your ideas!