switch our installed hicolor icons to the ones from breeze
ClosedPublic
Actions

Authored by sitter on Nov 22 2017, 11:07 AM.

Details

Reviewers

sandsmark
ltoscano

Commits

Summary

the old crystal icons are getting a bit long in the tooth and are
inconsistent on our windows build which use the breeze icons by default

Repository

R352 Filelight

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

also add 128x128 icon to get rid of cmake warning

No 128px icon provided; this will not work on Mac OS X

You probably need to fix also misc/CMakeLists.txt to install the 128x128 icon.

This revision now requires changes to proceed.Nov 22 2017, 11:40 AM

also install 128 into hicolor

The code is fine; I'm going to give a shipit, but not sure if you want to wait for @sandsmark 's ship it as well.

This revision is now accepted and ready to land.Nov 22 2017, 12:54 PM

This revision was automatically updated to reflect the committed changes.

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	22720	652fb80		Nov 22 2017, 11:07 AM	★	★
Diff 2	22725	652fb80	also add 128x128 icon to get rid of cmake warning	Nov 22 2017, 11:30 AM	★	★
Diff 3	22728	652fb80	also install 128 into hicolor	Nov 22 2017, 12:04 PM	★	★
Diff 4	23407	bb685f1	R352:078a6ef7c43f0f52f24c0acee220e3d0a22aa131	Dec 4 2017, 8:26 AM	★	★