Review Request: Limit tooltip size for klipper

Xuetian Weng wengxt at gmail.com
Sat Oct 13 16:29:42 UTC 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106829/
-----------------------------------------------------------

Review request for Plasma.


Description
-------

Klipper already limits the text size in the popup menu, however, it doesn't limit size of tooltip, which can still take up whole screen in some case.

There used to be another patch have similar idea: https://git.reviewboard.kde.org/r/103482/ , but I tried to use fontMetrics based API instead of hardcoded string length, though there is no easy way to obtain plasma tooltip font size and width is not possible from here (It's not really impossbile, but it will introduce more depenency, which is wanted).


Diffs
-----

  klipper/tray.cpp 3f4fc83 

Diff: http://git.reviewboard.kde.org/r/106829/diff/


Testing
-------

No problem with an acceptable size.


Thanks,

Xuetian Weng

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20121013/b73f52c7/attachment.html>


More information about the Plasma-devel mailing list