Review Request: The tooltip's tag are mismatch on krandrtray

Max Lin max7442 at gmail.com
Tue May 24 10:04:42 BST 2011


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

Review request for KDE Base Apps.


Summary
-------

After launch krandrtray, move cursor up on krandrtray, it will shows tooltip, however, looks the tag are not match so caused context are not aligned, and text color are not consistent, we also can catch the "tag mismatch" debug message. I added "<tr>" tag to match with "</tr>" tag, after with this patch, the "tag mismatch" message disappear.


Diffs
-----

  kcontrol/randr/krandrtray.cpp aab7295 

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


Testing
-------

Works fine on my openSUSE 11.4 with KDE 4.6.3 , I also attached screenshot before/after with this patch.


Screenshots
-----------

tag mismatch
  http://git.reviewboard.kde.org/r/101430/s/172/
tag match
  http://git.reviewboard.kde.org/r/101430/s/173/


Thanks,

Max

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20110524/5f5b1dbc/attachment.htm>


More information about the kde-core-devel mailing list