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