Add symbolic screenshot icons
Needs RevisionPublic

Authored by mbruchert on Thu, Feb 6, 6:26 PM.

Details

Reviewers
ngraham
lnj

Diff Detail

Repository
R266 Breeze Icons
Branch
sreenshot
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 22140
Build 22158: arc lint + arc unit
mbruchert created this revision.Thu, Feb 6, 6:26 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptThu, Feb 6, 6:26 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
mbruchert requested review of this revision.Thu, Feb 6, 6:26 PM
lnj added a comment.Sun, Feb 9, 2:38 PM

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.

lnj resigned from this revision.Sun, Feb 9, 2:38 PM
ngraham requested changes to this revision.Sun, Feb 9, 11:11 PM
ngraham added a subscriber: ngraham.

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

This revision now requires changes to proceed.Sun, Feb 9, 11:11 PM