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

(Updated July 15, 2012, 10:25 p.m.)


Review request for Telepathy.


Changes
-------

Addressed David's comments


Description
-------

This implements even smaller delegate for people who like to see looooots of 
contacts at once. I'm just not sure about using smallestReadableFont(), see 
screenshot - the generalFont() (on right) is good enough imho. Also the 
sizeHint() still should probably take font size into account.


This addresses bug 303051.
    http://bugs.kde.org/show_bug.cgi?id=303051


Diffs (updated)
-----

  contact-delegate-compact.h 7fa2907 
  contact-delegate-compact.cpp 948e5a8 
  contact-list-widget.h 724050d 
  contact-list-widget.cpp ee65883 
  main-widget.cpp 79869b8 

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


Testing
-------

Done.


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

Mini delegate, smallest font on left, general on right
  http://git.reviewboard.kde.org/r/105477/s/618/


Thanks,

Martin Klapetek

_______________________________________________
KDE-Telepathy mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kde-telepathy

Reply via email to