Use new video camera icon
ClosedPublic

Authored by ngraham on Fri, Feb 22, 8:53 PM.

Details

Summary

A new video camera icons was added in D18986. Let's use it in Kamoso!

This requires bumping the frameworks dependency to 5.56, since that's the release
that includes the new camera icon.

Test Plan

Diff Detail

Repository
R474 Kamoso
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ngraham requested review of this revision.Fri, Feb 22, 8:53 PM
ngraham created this revision.
filipf accepted this revision.Sat, Feb 23, 12:00 AM
filipf added a subscriber: filipf.

Looks good and is more appropriate than the old icon:

This revision is now accepted and ready to land.Sat, Feb 23, 12:00 AM
rooty accepted this revision.Sat, Feb 23, 12:33 AM

Thanks guys.

@apol, are you okay with bumping the KF5 dependency in order to use this icon?

apol accepted this revision.Sat, Feb 23, 11:48 PM

Removing it from the repository won't make it work for users who don't use breeze? LGTM in general.

Many other icon themes have this icon already. We were the late ones, in fact. :)

This revision was automatically updated to reflect the committed changes.