Bug 154054

Accepting the first part

The second part is prabably a Qt bug though, or you may be squeezing the widget yourself.

No, the second part is about using the widget inside a QTreeView (editing an item), so the size of the widget will be the size of the item. The height of the items are dependent on the height of the displayed text, and there isn't much spare room, so adding any type of frame more than one or possibly two pixels is going to cause problems.

That would make it a qt bug imo, but could you tell me where i can see this bug.

Preferably along with pointers to the the code.

SVN commit 754073 by boemann:

Fix focus in itemviews that have multiple selection.
BUG:154054


 M  +7 -0      oxygen.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=754073

Fredag 28 december 2007 19:23 skrev Casper Boemann:
[bugs.kde.org quoted mail]
I doubt *very* much the trolls will think it is a qt bug, but that just my 
opinion :)
You can see it in koffice/kplato (which I'm working on). I took a snapshot so 
you can see what it looks like.
>
> Preferably along with pointers to the the code.

The code that sets this up is in qitemdelegate.cpp:

void QItemDelegate::updateEditorGeometry(QWidget *editor,
                                         const QStyleOptionViewItem &option,
                                         const QModelIndex &index) const
{
    if (!editor)
        return;
    Q_ASSERT(index.isValid());
    QPixmap pixmap = decoration(option, index.data(Qt::DecorationRole));
    QString text = 
QItemDelegatePrivate::replaceNewLine(index.data(Qt::DisplayRole).toString());
    QRect pixmapRect = QRect(QPoint(0, 0), 
option.decorationSize).intersected(pixmap.rect());
    QRect textRect = textRectangle(0, option.rect, option.font, text);
    QRect checkRect = check(option, textRect, index.data(Qt::CheckStateRole));
    QStyleOptionViewItem opt = option;
    opt.showDecorationSelected = true; // let the editor take up all available 
space
    doLayout(opt, &checkRect, &pixmapRect, &textRect, false);
    editor->setGeometry(textRect);
}


Created an attachment (id=22748)
kplato.png

Just to clearify, it's not limited to oxygen/comboboxes.
It's the same with spinbox variants (doublespinbox, date/time edit...),
and the plastik style, which also draws a frame (a dotted line).
This isn't as obstructive as the oxygen style but still doesn't look good.

Lørdag 29 december 2007 01:41 skrev Casper Boemann:
[bugs.kde.org quoted mail]
This seems to draw a blue(ish) background which is almost the same as the 
color used for selected items and it is not possible to see which item has 
focus if it is also selected. (You may have a whole row selected while only 
one item has focus)
I think using a background color to identify focused item is (maybe) a good 
idea, it must contrast with selected item and it must have precedence over 
selected item as some operations targets focused item only.
>
>
>  M  +7 -0      oxygen.cpp
>
>
> WebSVN link: http://websvn.kde.org/?view=rev&revision=754073

SVN commit 754339 by boemann:

Do the focus rect in itemviews in a different way
Also make them show on top of selected items

CCBUG: 154054


 M  +12 -4     oxygen.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=754339