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