D29155: [Fonts KCM] Remove redundant nearestExistingFont()

Benjamin Port noreply at phabricator.kde.org
Mon May 4 16:43:12 BST 2020


bport added a comment.


  I don't think we will have same behavior, we don't only check name but also size, type...
  If we are ok to fallback in all case to the same font that can work.
  
  From your test plan, something look strange you don't end with a monospaced font for fixed font as fallback ?

REPOSITORY
  R119 Plasma Desktop

BRANCH
  l-setnearest (branched from master)

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

To: ahmadsamir, #plasma, bport, ngraham
Cc: ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, 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/20200504/05fdd7ef/attachment.html>


More information about the Plasma-devel mailing list