trmdi abandoned this revision. trmdi added a comment.
In D19096#427329 <https://phabricator.kde.org/D19096#427329>, @hein wrote: > I'm sorry, but I can't accept this patch as-is - the hoverArea-related changes are just wrong, it's not OK to couple a delegate to the view by making it set tons of property on an item i its parent. I'll need to set some time aside to analyze what you're trying to achieve there and propose an alternative. I just want 2 things: 1. Display 2 lines for long labels. 2. For labels need more than 2 lines -> display a tooltip that has the full label when hovering on it. That's all, no matter how you do it. Basically, I don't want to add any new feature, just want to make the existed stuff usable. (Currently it's not usable while it's existing) Thank you! REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D19096 To: trmdi, ngraham, hein, #plasma, #vdg, broulik Cc: abetts, ndavis, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart
