Differential D18785

Use different Font Management KCM "All Fonts" icon
ClosedPublic
Actions

Authored by GB_2 on Feb 6 2019, 12:45 PM.

Details

Reviewers
ngraham
Group Reviewers
Plasma
VDG
Commits
R119:e5ef9c97e75e: Use different Font Management KCM "All Fonts" icon
Summary

Needed for D18702.

Test Plan

Open the Font Management KCM.

Diff Detail

Repository
R119 Plasma Desktop
Branch
use-different-font-management-all-fonts-icon (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 7992
Build 8010: arc lint + arc unit
GB_2 created this revision.Feb 6 2019, 12:45 PM
Restricted Application added a project: Plasma. · View Herald TranscriptFeb 6 2019, 12:45 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
GB_2 requested review of this revision.Feb 6 2019, 12:45 PM
ngraham added a subscriber: ngraham.Feb 6 2019, 6:16 PM

The name makes me think this is not really semantically correct. Maybe we could make a symlink just called font or something that points to this icon?

Also this icon only has a 24px version so it would look blurry at the more common 16px and 24px sizes, as in fact your screenshot shows.

GB_2 updated this revision to Diff 51115.Feb 7 2019, 5:36 PM

Use new icon "font"

ngraham accepted this revision.Feb 7 2019, 5:38 PM
This revision is now accepted and ready to land.Feb 7 2019, 5:38 PM
Closed by commit R119:e5ef9c97e75e: Use different Font Management KCM "All Fonts" icon (authored by GB_2). · Explain WhyFeb 8 2019, 11:08 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 51115

View Options

kcms/kfontinst/kcmfontinst/GroupList.cpp

Loading...