Differential D3050

Use new PlasmaWindowInterface::setIcon
ClosedPublic
Actions

Authored by graesslin on Oct 13 2016, 11:58 AM.

Details

Reviewers
romangg
hein
Group Reviewers
Plasma on Wayland
KWin
Commits
R108:05d21b6b25ce: Use new PlasmaWindowInterface::setIcon
Summary

This allows to pass also Xwayland icons to the task manager.

Diff Detail

Repository
R108 KWin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
graesslin updated this revision to Diff 7377.Oct 13 2016, 11:58 AM
graesslin retitled this revision from to Use new PlasmaWindowInterface::setIcon.
graesslin updated this object.
graesslin edited the test plan for this revision. (Show Details)
graesslin added reviewers: KWin, Plasma on Wayland, hein.
Restricted Application added projects: Plasma on Wayland, KWin. · View Herald TranscriptOct 13 2016, 11:58 AM
Restricted Application added subscribers: kwin, plasma-devel. · View Herald Transcript
romangg accepted this revision.Oct 16 2016, 11:57 AM
romangg added a reviewer: romangg.
romangg added a subscriber: romangg.

Tested with Inkscape, Chrome, Gimp. Works fine, for Gimp (tested only in single-window mode) it needed some time to show the app/icon in the taskbar, but the delay in showing a taskbar entry for Gimp happens also without this patch on Wayland.

This revision is now accepted and ready to land.Oct 16 2016, 11:57 AM
Closed by commit R108:05d21b6b25ce: Use new PlasmaWindowInterface::setIcon (authored by graesslin). · Explain WhyOct 17 2016, 7:03 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mabstract_client.cpp (5 lines)

Diff 7447

View Options

abstract_client.cpp

Loading...