D28274: [KFontChooser] Add a checkbox to toggle showing only monospaced fonts
David Faure
noreply at phabricator.kde.org
Sat Apr 11 15:10:52 BST 2020
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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200411/91722a4f/attachment.html>
More information about the Kde-frameworks-devel
mailing list