Differential D21429

[Fonts KCM] Switch default UI font to Noto Sans Display
AbandonedPublic
Actions

Authored by GB_2 on May 27 2019, 3:14 PM.

Details

Reviewers
bshah
Group Reviewers
Plasma
VDG
Maniphest Tasks
T10980: Switch default UI font to Noto Sans Display
Summary

See T10980 for more info.

Test Plan

Noto Sans Display should now be the default font.

Diff Detail

Repository
R119 Plasma Desktop
Branch
switch-default-ui-font-to-noto-sans-display (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 12193
Build 12211: arc lint + arc unit
GB_2 created this revision.May 27 2019, 3:14 PM
Restricted Application added a project: Plasma. · View Herald TranscriptMay 27 2019, 3:14 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
GB_2 requested review of this revision.May 27 2019, 3:14 PM
bshah requested changes to this revision.May 27 2019, 3:22 PM
bshah added a subscriber: bshah.

Please make sure you also update kcms/krdb/krdb.cpp as well ^^

This revision now requires changes to proceed.May 27 2019, 3:22 PM
GB_2 updated this revision to Diff 58730.May 27 2019, 3:26 PM

Also update KRDB

GB_2 added a comment.May 27 2019, 5:29 PM
In D21429#470715, @hein wrote:

Why?

Read the task mentioned in the description...

See T10980 for more info.

hein added a comment.May 27 2019, 5:35 PM

I'm not sold on this. "Display" fonts typically have spacing and metrics optimized for large sizes and use cases like signage. The FAQ entry quoted in the task you linked talks about Noto Sans UI. I'd like some more solid documentation on how Display relates to this "UI" cut or something from upstream.

GB_2 abandoned this revision.May 28 2019, 5:53 AM

Revision Contents
Changeset List

PathPackages
Mkcms/fonts/fonts.cpp (10 lines)
Mkcms/krdb/krdb.cpp (2 lines)

Diff 58730

View Options

kcms/fonts/fonts.cpp

Loading...
View Options

kcms/krdb/krdb.cpp

Loading...