dfaure accepted this revision.
dfaure added inline comments.
INLINE COMMENTS
> kfontchooser.cpp:393
> q->setFont(QFontDatabase::systemFont(QFontDatabase::FixedFont),
> usingFixed);
> + onlyFixedCheckbox->setChecked(usingFixed);
> } else {
would `true` be more readable?
(same for the previous line, actually)
REPOSITORY
R236 KWidgetsAddons
BRANCH
l-monospace-checkbox (branched from master)
REVISION DETAIL
https://phabricator.kde.org/D28274
To: ahmadsamir, #frameworks, dfaure, cfeck, apol, bport
Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns
