D29155: [Fonts KCM] Remove redundant nearestExistingFont()

Ahmad Samir noreply at phabricator.kde.org
Tue May 5 12:26:33 BST 2020


ahmadsamir added a comment.


  In D29155#663178 <https://phabricator.kde.org/D29155#663178>, @bport wrote:
  
  > 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 ?
  
  
  What happens currently is it selects "Sans Serif".

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/20200505/9516d187/attachment.html>


More information about the Plasma-devel mailing list