Change terminal panel icon to dialog-scripts
ClosedPublic

Authored by ndavis on Thu, Sep 5, 3:39 PM.

Details

Summary

The old icon was a color icon, which does not match the style of other toolbar/menu buttons

Test Plan

Diff Detail

Repository
R318 Dolphin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ndavis created this revision.Thu, Sep 5, 3:39 PM
Restricted Application added a project: Dolphin. · View Herald TranscriptThu, Sep 5, 3:39 PM
Restricted Application added a subscriber: kfm-devel. · View Herald Transcript
ndavis requested review of this revision.Thu, Sep 5, 3:39 PM
ndavis edited the test plan for this revision. (Show Details)Thu, Sep 5, 3:40 PM
ndavis edited the test plan for this revision. (Show Details)
ndavis edited the test plan for this revision. (Show Details)
ngraham added a subscriber: ngraham.Thu, Sep 5, 3:46 PM

Definitely better to use a real monochrome action icon for this. Looks better to me, in conjunction with the Breeze icons change.

ngraham accepted this revision.Thu, Sep 5, 3:47 PM
This revision is now accepted and ready to land.Thu, Sep 5, 3:47 PM
This revision was automatically updated to reflect the committed changes.