D6544: Don't draw focus line on selected list item

Christoph Feck noreply at phabricator.kde.org
Fri Jul 7 12:12:49 UTC 2017


cfeck added inline comments.

INLINE COMMENTS

> cfeck wrote in breezestyle.cpp:3082
> Of course inherits(QString) is slower than qobject_cast<>.
> 
> The only reason to use inherits() is when you do not link to the library that the class defines.

Also order to make the fastest test come first.

if ((state & State_Selected) && qobject_cast<QAbstractItemView *>(widget))

REPOSITORY
  R31 Breeze

REVISION DETAIL
  https://phabricator.kde.org/D6544

To: broulik, #plasma, hpereiradacosta, fvogt
Cc: cfeck, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170707/e4ea0134/attachment.html>


More information about the Plasma-devel mailing list