D27416: Add small icons for KeePassXC

2020-03-17 Thread Nathaniel Graham 
This revision was automatically updated to reflect the committed changes.
Closed by commit R266:036ba91a2bc3: Add small icons for KeePassXC (authored by 
wsdfhjxc, committed by ngraham).

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27416?vs=77809=77854

REVISION DETAIL
  https://phabricator.kde.org/D27416

AFFECTED FILES
  icons-dark/status/22/keepassxc-locked.svg
  icons-dark/status/22/keepassxc-unlocked.svg
  icons/status/22/keepassxc-locked.svg
  icons/status/22/keepassxc-unlocked.svg

To: wsdfhjxc, #vdg, cblack, ndavis
Cc: ndavis, cblack, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, 
bruns

D27416: Add small icons for KeePassXC

2020-03-17 Thread Carson Black 
cblack accepted this revision.
cblack added a comment.
This revision is now accepted and ready to land.


  LGTM now. Good icon 

REPOSITORY
  R266 Breeze Icons

BRANCH
  keepassxc-small

REVISION DETAIL
  https://phabricator.kde.org/D27416

To: wsdfhjxc, #vdg, cblack, ndavis
Cc: ndavis, cblack, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, 
bruns

D27416: Add small icons for KeePassXC

2020-03-17 Thread Noah Davis 
ndavis accepted this revision.
ndavis added a comment.


  LGTM

REPOSITORY
  R266 Breeze Icons

REVISION DETAIL
  https://phabricator.kde.org/D27416

To: wsdfhjxc, #vdg, cblack, ndavis
Cc: ndavis, cblack, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, 
bruns

D27416: Add small icons for KeePassXC

2020-03-17 Thread Wojciech Stachurski 
wsdfhjxc added a comment.


  Is it any better now? @cblack @ndavis

REPOSITORY
  R266 Breeze Icons

REVISION DETAIL
  https://phabricator.kde.org/D27416

To: wsdfhjxc, #vdg, cblack, ndavis
Cc: ndavis, cblack, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, 
bruns

D27416: Add small icons for KeePassXC

2020-03-17 Thread Wojciech Stachurski 
wsdfhjxc edited the summary of this revision.

REPOSITORY
  R266 Breeze Icons

REVISION DETAIL
  https://phabricator.kde.org/D27416

To: wsdfhjxc, #vdg, cblack, ndavis
Cc: ndavis, cblack, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, 
bruns

D27416: Add small icons for KeePassXC

2020-03-17 Thread Wojciech Stachurski 
wsdfhjxc edited the summary of this revision.

REPOSITORY
  R266 Breeze Icons

REVISION DETAIL
  https://phabricator.kde.org/D27416

To: wsdfhjxc, #vdg, cblack, ndavis
Cc: ndavis, cblack, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, 
bruns

D27416: Add small icons for KeePassXC

2020-03-17 Thread Wojciech Stachurski 
wsdfhjxc updated this revision to Diff 77809.
wsdfhjxc added a comment.


  Update small icons for KeePassXC

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27416?vs=75722=77809

BRANCH
  keepassxc-small

REVISION DETAIL
  https://phabricator.kde.org/D27416

AFFECTED FILES
  icons-dark/status/22/keepassxc-locked.svg
  icons-dark/status/22/keepassxc-unlocked.svg
  icons/status/22/keepassxc-locked.svg
  icons/status/22/keepassxc-unlocked.svg

To: wsdfhjxc, #vdg, cblack, ndavis
Cc: ndavis, cblack, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, 
bruns

D27416: Add small icons for KeePassXC

2020-03-16 Thread Noah Davis 
ndavis requested changes to this revision.
ndavis added a comment.


  Since this is a systray icon, you don't actually need to do the 16px version, 
but I guess there's no harm in having a 16px version.
  
  As @cblack said, this needs some work on the pixel alignment. If you don't do 
the 16px version, it'll be significantly less work.
  
  The orange X should be a lock emblem instead. See the `firewall-applet-panic` 
icon.

REPOSITORY
  R266 Breeze Icons

REVISION DETAIL
  https://phabricator.kde.org/D27416

To: wsdfhjxc, #vdg, cblack, ndavis
Cc: ndavis, cblack, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, 
bruns

D27416: Add small icons for KeePassXC

2020-03-16 Thread Carson Black 
cblack requested changes to this revision.
cblack added a comment.
This revision now requires changes to proceed.


  The key in both icons should be pixel aligned to prevent blurry edges at the 
size it's going to be rendered at.
  
  F8180107: image.png 
  
  This should also be in `status/` instead of `apps/` as it's a tray icon.

REPOSITORY
  R266 Breeze Icons

REVISION DETAIL
  https://phabricator.kde.org/D27416

To: wsdfhjxc, #vdg, cblack
Cc: cblack, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D27416: Add small icons for KeePassXC

2020-02-14 Thread Wojciech Stachurski 
wsdfhjxc edited the summary of this revision.

REPOSITORY
  R266 Breeze Icons

REVISION DETAIL
  https://phabricator.kde.org/D27416

To: wsdfhjxc, #vdg
Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D27416: Add small icons for KeePassXC

2020-02-14 Thread Wojciech Stachurski 
wsdfhjxc created this revision.
wsdfhjxc added a reviewer: VDG.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
wsdfhjxc requested review of this revision.

REPOSITORY
  R266 Breeze Icons

BRANCH
  keepassxc-small

REVISION DETAIL
  https://phabricator.kde.org/D27416

AFFECTED FILES
  icons-dark/apps/16/keepassxc-locked.svg
  icons-dark/apps/16/keepassxc-unlocked.svg
  icons-dark/apps/22/keepassxc-locked.svg
  icons-dark/apps/22/keepassxc-unlocked.svg
  icons/apps/16/keepassxc-locked.svg
  icons/apps/16/keepassxc-unlocked.svg
  icons/apps/22/keepassxc-locked.svg
  icons/apps/22/keepassxc-unlocked.svg

To: wsdfhjxc, #vdg
Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns