D27197: Add symbolic screenshot icons
ngraham added a comment. Thanks, that's better now. However the camera silhouette still doesn't match that of other monochrome camera images we have. I'd like to stick to standard iconography if possible. Do you think we should switch the other related icons to use your new camera icon, or would you like to use the one from, for example, `preferences-system-windows-effect-screenshot`? REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D27197 To: mbruchert, ngraham Cc: ngraham, kde-frameworks-devel, LeGast00n, cblack, michaelh, bruns
D27197: Add symbolic screenshot icons
mbruchert updated this revision to Diff 83320. mbruchert added a comment. Fix colors REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27197?vs=75125&id=83320 BRANCH sreenshot REVISION DETAIL https://phabricator.kde.org/D27197 AFFECTED FILES icons-dark/actions/16/screenshot.svg icons-dark/actions/22/screenshot.svg icons/actions/16/screenshot.svg icons/actions/22/screenshot.svg To: mbruchert, ngraham Cc: ngraham, kde-frameworks-devel, LeGast00n, cblack, michaelh, bruns
D27197: Add symbolic screenshot icons
ngraham requested changes to this revision. ngraham added a comment. This revision now requires changes to proceed. Please run the icons through an SVG optimizer and remove the `color=` attributes inside the paths. See https://community.kde.org/Guidelines_and_HOWTOs/Icon_Workflow_Tips for more details. Also, please add Breeze Dark versions in the `icons-dark` folder. Finally, in terms of design, shouldn't the camera in this icon look like the camera used in other icons? For example `camera-photo-symbolic` REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D27197 To: mbruchert, ngraham Cc: ngraham, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, bruns
D27197: Add symbolic screenshot icons
lnj resigned from this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D27197 To: mbruchert Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns
D27197: Add symbolic screenshot icons
lnj added a comment. This is required for the screenshot action in the panel of the plasma mobile shell. I don't know enough about creating breeze icons to approve this diff, but at least I don't see any problems. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D27197 To: mbruchert, lnj Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns
D27197: Add symbolic screenshot icons
mbruchert added a reviewer: lnj. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D27197 To: mbruchert, lnj Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns
D27197: Add symbolic screenshot icons
mbruchert created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. mbruchert requested review of this revision. REPOSITORY R266 Breeze Icons BRANCH sreenshot REVISION DETAIL https://phabricator.kde.org/D27197 AFFECTED FILES icons/actions/16/screenshot.svg icons/actions/22/screenshot.svg To: mbruchert Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns