D27808: [Fonts KCM] Port KFontDialog/KFontChooser to QFontDialog

Ahmad Samir noreply at phabricator.kde.org
Tue Mar 3 16:34:54 GMT 2020


ahmadsamir created this revision.
ahmadsamir added reviewers: Plasma, davidedmundson, broulik, ervin, meven.
Herald added a project: Plasma.
ahmadsamir requested review of this revision.

REVISION SUMMARY
  When using ajustAllFonts(), try to set the fixed width/Monospace font
  from the font family the user has selected, otherwise fallback to whatever
  QFontDatabase::systemFont(QFontDatabase::FixedFont) gets.
  
  Depends on D27785 <https://phabricator.kde.org/D27785>

TEST PLAN
  Changing a font still works.
  Due to the change to clear font styleName property when saving via
  KConfig (D27735 <https://phabricator.kde.org/D27735>), if in kdegloblas you removed the ",Regular|Normal|Book|Roman"
  from the end of the *font*=  entries, openin the font selection dialog
  in the KCM, the very first style is selected, not "Regular" and co. as
  it should be. This will be addressed in an upcoming diff.

REPOSITORY
  R119 Plasma Desktop

BRANCH
  l-qfontdlg-track-nearest (branched from master)

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

AFFECTED FILES
  kcms/fonts/fonts.cpp
  kcms/fonts/fonts.h

To: ahmadsamir, #plasma, davidedmundson, broulik, ervin, meven
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200303/ac136539/attachment.html>


More information about the Plasma-devel mailing list