D26266: Change remove action's icon to "edit-delete"

2019-12-29 Thread Tranter Madi
trmdi abandoned this revision. trmdi added a comment. In D26266#584135 , @mvourlakos wrote: > What is ConfigOverlay for plasma panels is using at its tooltip? Ah, I was wrong. "delete" should be the right one. Sorry.

D26266: Change remove action's icon to "edit-delete"

2019-12-29 Thread Michail Vourlakos
mvourlakos added a comment. What is ConfigOverlay for plasma panels is using at its tooltip? REPOSITORY R878 Latte Dock REVISION DETAIL https://phabricator.kde.org/D26266 To: trmdi, #latte_dock, #vdg, mvourlakos Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus,

D26266: Change remove action's icon to "edit-delete"

2019-12-29 Thread Tranter Madi
trmdi added a comment. But I've just found that Plasma panels use QIcon::fromTheme which load the icon from the icon theme, while PlasmaComponent.ToolButton loads icon from Plasma theme, that is not consistent in some cases. What do you think? REPOSITORY R878 Latte Dock BRANCH master

D26266: Change remove action's icon to "edit-delete"

2019-12-28 Thread Tranter Madi
trmdi created this revision. trmdi added reviewers: Latte Dock, VDG. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. trmdi requested review of this revision. REVISION SUMMARY Follow Plasma codes