D29065: [KFontChooser] Add a DisplayFlag to allow not showing fixedOnly checkbox

[David Faure]

dfaure requested changes to this revision.
dfaure added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> kfontchooser.h:90
> +        ShowDifferences = 4,    ///< Display the font differences interfaces
> +        NoFixedOnlyToggle = 5   ///< Show a checkbox to toggle showing only fixed fonts
>      };

1/2/4 looked like powers of 2, i.e. a flag set.

Using 5 here gives is the same value as FixedFontsOnly | ShowDifferences.

On the other hand it makes sense FixedFontsOnly would automatically hide the checkbox, no? If the app says "fixed fonts only", the user has no choice in the matter.

I'm also confused by the name "No[...]Toggle" and the docu saying this will show a checkbox. Something's off by "not" ;)
Sounds wrong?

REPOSITORY
  R236 KWidgetsAddons

REVISION DETAIL
  https://phabricator.kde.org/D29065

To: ahmadsamir, #frameworks, dfaure, cfeck, bport
Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200422/fd07946d/attachment.html>