D14105: Use a broom-style icon for clearing clipboard and notification history

2018-07-23 Thread Nathaniel Graham
ngraham added a comment. ``Check out https://community.kde.org/Get_Involved/development#Get_the_code :)`` REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D14105 To: ngraham, broulik, Fuchs, #plasma, davidedmundson Cc: abetts, Zren, plasma-devel, ragreen, Pi

D14105: Use a broom-style icon for clearing clipboard and notification history

2018-07-23 Thread Andres Betts
abetts added a comment. In D14105#296525 , @ngraham wrote: > It's the `edit-clear-history` icon in the `breeze-icons` repo. Can you help me find that repo please? I have no clue how to get there. REPOSITORY R120 Plasma Workspace REVISI

D14105: Use a broom-style icon for clearing clipboard and notification history

2018-07-23 Thread Nathaniel Graham
ngraham added a comment. It's the `edit-clear-history` icon in the `breeze-icons` repo. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D14105 To: ngraham, broulik, Fuchs, #plasma, davidedmundson Cc: abetts, Zren, plasma-devel, ragreen, Pitel, ZrenBot, leslie

D14105: Use a broom-style icon for clearing clipboard and notification history

2018-07-23 Thread Andres Betts
abetts added a comment. Can you share the svg file for this icon please? I would like to edit it a little bit REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D14105 To: ngraham, broulik, Fuchs, #plasma, davidedmundson Cc: abetts, Zren, plasma-devel, ragreen,

D14105: Use a broom-style icon for clearing clipboard and notification history

2018-07-15 Thread Nathaniel Graham
ngraham closed this revision. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D14105 To: ngraham, broulik, Fuchs, #plasma, davidedmundson Cc: Zren, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D14105: Use a broom-style icon for clearing clipboard and notification history

2018-07-15 Thread Christian
Fuchs accepted this revision. Fuchs added a comment. if it falls back then all is perfect :) REPOSITORY R120 Plasma Workspace BRANCH broom-style-clear-history-icon (branched from master) REVISION DETAIL https://phabricator.kde.org/D14105 To: ngraham, broulik, Fuchs, #plasma, davidedmu

D14105: Use a broom-style icon for clearing clipboard and notification history

2018-07-15 Thread Nathaniel Graham
ngraham added a comment. Right, it's not a problem: 1. Either the theme has its own version of that icon (as Oxygen does) 2. ...Or else the theme symlinks that icon to something else (as Papirus does; it's symlinked to `edit-clear` which looks like the backspace symbol)... 3. ...Or el

D14105: Use a broom-style icon for clearing clipboard and notification history

2018-07-15 Thread Nathaniel Graham
ngraham added a comment. It't not a problem: either theme theme has its own version of that icon, or else it falls back to the Breeze icon, which in 5.48 is a broom, and prior to 5.48 looks like the backspace symbol. REPOSITORY R120 Plasma Workspace BRANCH broom-style-clear-history-icon

D14105: Use a broom-style icon for clearing clipboard and notification history

2018-07-15 Thread Christian
Fuchs added a comment. In D14105#292183 , @ngraham wrote: > Thanks! @fuchs and/or @broulik? Can't test it currently, looks fine, just one question: did you test what happens on other icon sets than breeze? If that doesn't work, I'd say

D14105: Use a broom-style icon for clearing clipboard and notification history

2018-07-14 Thread Nathaniel Graham
ngraham added a comment. Thanks! @fuchs and/or @broulik? REPOSITORY R120 Plasma Workspace BRANCH broom-style-clear-history-icon (branched from master) REVISION DETAIL https://phabricator.kde.org/D14105 To: ngraham, broulik, Fuchs, #plasma, davidedmundson Cc: Zren, plasma-devel, ragree

D14105: Use a broom-style icon for clearing clipboard and notification history

2018-07-14 Thread Nathaniel Graham
ngraham updated this revision to Diff 37745. ngraham added a comment. Also use the new icon in the context menu REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D14105?vs=37717&id=37745 BRANCH broom-style-clear-history-icon (branched from master)

D14105: Use a broom-style icon for clearing clipboard and notification history

2018-07-13 Thread Chris Holland
Zren added a comment. Should the context menu also use this icon? REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D14105 To: ngraham, broulik, Fuchs, #plasma Cc: Zren, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, se

D14105: Use a broom-style icon for clearing clipboard and notification history

2018-07-13 Thread Nathaniel Graham
ngraham created this revision. ngraham added reviewers: broulik, Fuchs, Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. ngraham requested review of this revision. REVISION SUMMARY Use Breeze's new broom-style icon for the Clear His