ndavis added a comment.
In D19633#427913 <https://phabricator.kde.org/D19633#427913>, @ngraham wrote: > Neat! Very helpful, thanks. > > While you're at it, do you think you could add a `window-unpin` icon too? https://bugs.kde.org/show_bug.cgi?id=405195 Yes, but I'm not sure if it should copy the style of the unpin button from the tilebar or if it should have a line through it like you suggested. I feel like the line is more clear, but less consistent. Maybe it's the titlebar button unpin that should be changed? In D19633#427915 <https://phabricator.kde.org/D19633#427915>, @ngraham wrote: > Also `window-shade`? That can be done once D19575 <https://phabricator.kde.org/D19575> and D19638 <https://phabricator.kde.org/D19638> are landed. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D19633 To: ndavis, #vdg Cc: ngraham, kde-frameworks-devel, michaelh, bruns
