subdiff accepted this revision.
subdiff added a reviewer: subdiff.
subdiff added a comment.


  Told you, I had problems compiling at first. Works now for unknown reason and 
icons are displayed together with https://phabricator.kde.org/D3050.

INLINE COMMENTS

> graesslin wrote in plasmawindowmanagement_interface.cpp:386
> in theory yes, in practice: operator== doesn't exist for QIcon so all you get 
> is a compile error :-(

What about comparing QIcon::cacheKey() values?

REPOSITORY
  rKWAYLAND KWayland

BRANCH
  plasma-window-icon

REVISION DETAIL
  https://phabricator.kde.org/D3049

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #plasma_on_wayland, hein, roberts, subdiff
Cc: subdiff, apol, roberts, plasma-devel, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas

Reply via email to