ahmadsamir marked an inline comment as done. ahmadsamir added inline comments.
INLINE COMMENTS > dfaure wrote in kfontchooser.cpp:393 > would `true` be more readable? > > (same for the previous line, actually) Indeed; however onlyFixedCheckbox->setChecked(usingFixed) doesn't need to be in that if block at all. 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, michaelh, ngraham, bruns
