Add virt-manager icons
ClosedPublic
Actions

Authored by ndavis on Apr 1 2018, 3:47 PM.

Details

Reviewers

Group Reviewers

VDG
Breeze

Summary

48px color and 22px, 16px monochrome icons for the system tray

Repository

R266 Breeze Icons

Branch

virt-manager_icons (branched from master)

Lint

No Linters Available

Unit

No Unit Test Coverage

Restricted Application added a subscriber: Frameworks. · View Herald Transcript

ndavis requested review of this revision.Apr 1 2018, 3:47 PM

Screenshot of how they look in use

Looks good to me! Nice use of the Breeze icon style. @andreaska?

This revision is now accepted and ready to land.Apr 1 2018, 5:20 PM

In D11863#238210, @ngraham wrote:

Looks good to me! Nice use of the Breeze icon style. @andreaska?

Thanks! If the other reviewer takes a long time to respond, do I still have to wait for their permission to land it?

Let's cross that bridge when we come to it! :-)

Since I don't have a developer account, can someone land this for me?

Thanks for the commit.

I checked the icons and optimize the icon size. virt-manager is now in master.

		Path
A	M	icons-dark/apps/16/virt-manager.svg (143 lines)
A	M	icons-dark/apps/22/virt-manager.svg (107 lines)
A	M	icons-dark/apps/48/virt-manager.svg (400 lines)
A	M	icons/apps/16/virt-manager.svg (143 lines)
A	M	icons/apps/22/virt-manager.svg (107 lines)
A	M	icons/apps/48/virt-manager.svg (400 lines)