D4917: [Breeze Plasma Theme] Improve action-overlay icons

broulik added a comment. > only a draft based on the old design with the new semantic. The graphic is missing some states (add-hover, add-pressed, open-normal, open-hovered, remove-hover, remove-pressed). Not a fan of having a blue remove icon. Also, I was told the arrow isn't a

D4917: [Breeze Plasma Theme] Improve action-overlay icons

andreaska added a comment. F2753390: action-overlays.svgz only a draft (not finished, only that you can review something) F2753393: overlayScreenshot.png REPOSITORY R242 Plasma Framework (Library) REVISION

D4917: [Breeze Plasma Theme] Improve action-overlay icons

broulik added a comment. The icons are copied from the Breeze icon set. Granted, it's an "emblem" icon and not one from "actions" which is arguably wrong but I wanted to be consistent with Dolphin here… also, I expressed my frustration with our current visual design workflow numerous times,

D4917: [Breeze Plasma Theme] Improve action-overlay icons

hein added a comment. I wouldn't have veto'd it, but FWIW I think those icons don't work - their stem strength is much wider than Breeze lineart icons, they feel very out of place in our UX. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D4917

D4917: [Breeze Plasma Theme] Improve action-overlay icons

This revision was automatically updated to reflect the committed changes. Closed by commit R242:badf8acfe2c0: [Breeze Plasma Theme] Improve action-overlay icons (authored by broulik). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE

D4917: [Breeze Plasma Theme] Improve action-overlay icons

mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D4917 To: broulik, #plasma, #vdg, andreask, alex-l, mart Cc: plasma-devel, #frameworks, progwolff, lesliezhai, ali-mohamed,

D4917: [Breeze Plasma Theme] Improve action-overlay icons

broulik edited the test plan for this revision. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D4917 To: broulik, #plasma, #vdg, andreask, alex-l Cc: plasma-devel, #frameworks, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol

D4917: [Breeze Plasma Theme] Improve action-overlay icons

broulik added reviewers: andreask, alex-l. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D4917 To: broulik, #plasma, #vdg, andreask, alex-l Cc: plasma-devel, #frameworks, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol

D4917: [Breeze Plasma Theme] Improve action-overlay icons

broulik edited the test plan for this revision. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D4917 To: broulik, #plasma, #vdg Cc: plasma-devel, #frameworks, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol

D4917: [Breeze Plasma Theme] Improve action-overlay icons

broulik created this revision. Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel. REVISION SUMMARY The current ones aren't really descriptive and vastly different from what Dolphin uses. BUG: 376321 TEST PLAN