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

Hugo Pereira Da Costa noreply at phabricator.kde.org
Fri Jul 7 09:55:20 UTC 2017


hpereiradacosta added inline comments.

INLINE COMMENTS

> breezestyle.cpp:3082
> +        // no focus indicator on selected list items
> +        if (widget && widget->inherits("QAbstractItemView") && (state & State_Selected))
> +        { return true; }

Not sure which one is the most efficient:
Widget->inherits or qobject_cast ?
(My guess  would need the second, but ...)
Can you double check ?

REPOSITORY
  R31 Breeze

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

To: broulik, #plasma, hpereiradacosta, fvogt
Cc: 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/6aa9df65/attachment-0001.html>


More information about the Plasma-devel mailing list