Add edit-line-width icons
ClosedPublic

Authored by trickyricky26 on Oct 4 2019, 12:47 PM.

Details

Reviewers
ndavis
Group Reviewers
VDG
Commits
R266:2b522d49a7aa: Add edit-line-width icons
Summary

Add edit-line-width icons required for T8076.

CCBUG:408283

Test Plan

Diff Detail

Repository
R266 Breeze Icons
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
trickyricky26 created this revision.Oct 4 2019, 12:47 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptOct 4 2019, 12:47 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
trickyricky26 requested review of this revision.Oct 4 2019, 12:47 PM
trickyricky26 edited the summary of this revision. (Show Details)Oct 4 2019, 12:49 PM
trickyricky26 edited the test plan for this revision. (Show Details)
trickyricky26 added a reviewer: VDG.
ndavis added a subscriber: ndavis.Oct 4 2019, 6:37 PM

I think something similar to this line width icon would work well:


Inkscape uses a similar icon, but for all line styles:

trickyricky26 planned changes to this revision.Oct 5 2019, 10:18 AM

I think something similar to this line width icon would work well:


Inkscape uses a similar icon, but for all line styles:

Yeah, that seems like a more obvious choice.

  • Use design with multiple lines of different thickness
trickyricky26 edited the test plan for this revision. (Show Details)Oct 5 2019, 2:31 PM
ngraham added a subscriber: ngraham.Oct 5 2019, 3:27 PM

Much better!

trickyricky26 edited the summary of this revision. (Show Details)Oct 5 2019, 4:08 PM
ndavis accepted this revision.Oct 5 2019, 7:54 PM
This revision is now accepted and ready to land.Oct 5 2019, 7:54 PM
Closed by commit R266:2b522d49a7aa: Add edit-line-width icons (authored by Rafael Brandmaier <rafael.brandmaier@kdemail.net>). · Explain WhyOct 6 2019, 5:07 PM
This revision was automatically updated to reflect the committed changes.