ahmadsamir added inline comments.
INLINE COMMENTS
> fonts.cpp:271
> + // tiny, they want a tiny font everywhere.
> + if (font.pointSize() > 8) {
> + smallestFont.setPointSize(font.pointSize() - 2);
Nit-pick: I'd store font.pointSize() in a const int, and use that instead of
calling pointSize() more than once.
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D29156
To: ngraham, #plasma, ahmadsamir, bport
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus,
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed,
jensreuterberg, abetts, sebas, apol, ahiemstra, mart
