[breeze-icons] add TeamViewer tray icons
ClosedPublic

Authored by rocka on Tue, Mar 17, 9:56 AM.

Details

Summary

Created TeamViewer tray icons following Breeze style.

Test Plan

Before:

After:

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.
rocka created this revision.Tue, Mar 17, 9:56 AM
Restricted Application added a project: Frameworks. · View Herald TranscriptTue, Mar 17, 9:56 AM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
rocka requested review of this revision.Tue, Mar 17, 9:56 AM
rocka edited the test plan for this revision. (Show Details)Tue, Mar 17, 9:57 AM
rocka added a reviewer: VDG.
ngraham accepted this revision.Tue, Mar 17, 4:26 PM
ngraham added subscribers: ndavis, ngraham.

LGTM. @ndavis?

This revision is now accepted and ready to land.Tue, Mar 17, 4:26 PM
ndavis accepted this revision.Tue, Mar 17, 5:33 PM
This revision was automatically updated to reflect the committed changes.

I didn't notice it before, but this patch makes TeamViewer's app icon always use the monochrome version now:

We already have a Breeze icon for TeamViewer called teamviewer note the capitalization. It seems like somewhere in our stack, teamviewer and TeamViewer get mixed up such that everything uses the monochrome icon.

rocka added a comment.Thu, Mar 19, 1:33 AM

Could we just create a symlink TeamViewer.svg points to 48px teamviewer.svg? And I noticed that in their latest version, they changed the Icon in desktop entry file to TeamViewer, and all the installed icons were named TeamViewer.

After creating that symlink, the icon works fine in Task Manager and Application Dashboard, but still looks monochrome in Application Launcher and Application Menu.

That will help for people using thick panels who see larger icons, but users of thin panels will still get the monochrome icon.

It's yet another example of our inability to deterministically T10413: Find a way to specify whether to use monochrome or color icons in applications. :/

That said, TeamViewer changing the icon means that our current teamviewer icon isn't used at all, so I guess renaming its larger sizes to match the same name as the system tray icon would be better than nothing.