Review Request 129836: Use custom font im KMail message list

[Gerd Fleischer]

> On Jan. 16, 2017, 6:52 a.m., Luca Beltrame wrote:
> > Did you measure the performance impact? Most of these options were removed because they had a *huge* impact when displaying the message list view.

This method is only called on startup and if the font is changed in the config dialog, so it should not have any impact as the font is then cached.


- Gerd


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129836/#review102055
-----------------------------------------------------------


On Jan. 15, 2017, 11:24 a.m., Gerd Fleischer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129836/
> -----------------------------------------------------------
> 
> (Updated Jan. 15, 2017, 11:24 a.m.)
> 
> 
> Review request for KDEPIM.
> 
> 
> Bugs: 362803
>     https://bugs.kde.org/show_bug.cgi?id=362803
> 
> 
> Repository: messagelib
> 
> 
> Description
> -------
> 
> Use custom font in KMail Messagelist if checked,
> currently the systemfont is always used.
> 
> This is related to BUG 362803
> 
> 
> Diffs
> -----
> 
>   messagelist/src/core/themedelegate.cpp 9b71bfc 
> 
> Diff: https://git.reviewboard.kde.org/r/129836/diff/
> 
> 
> Testing
> -------
> 
> Switching custom font and size in KMail message list works for me now.
> 
> 
> Thanks,
> 
> Gerd Fleischer
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20170116/79d32f75/attachment.html>