D28093: [breeze-icons] add TeamViewer tray icons

2020-03-18 Thread Rocket Aaron 
rocka added a comment.


  Could we just create a symlink TeamViewer.svg points to 48px teamviewer.svg? 
And I noticed that in their latest version, they changed the Icon in desktop 
entry file to TeamViewer, and all the installed icons were named TeamViewer.
  
  After creating that symlink, the icon works fine in Task Manager and 
Application Dashboard, but still looks monochrome in Application Launcher and 
Application Menu.
  
  F8183756: a.png 

REPOSITORY
  R266 Breeze Icons

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

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

D28094: [breeze-icons] fix shadow in some user (applets/128) icons

2020-03-18 Thread Rocket Aaron 
rocka updated this revision to Diff 77886.
rocka added a comment.


  Try to fix icon display in inkscape

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D28094?vs=77814=77886

BRANCH
  fix-user-icon-shadow_2 (branched from master)

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

AFFECTED FILES
  icons-dark/applets/128/user-ada-lovelace.svg
  icons-dark/applets/128/user-leonardo-da-vinci.svg
  icons/applets/128/user-ada-lovelace.svg
  icons/applets/128/user-leonardo-da-vinci.svg

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

D28094: [breeze-icons] fix shadow in some user (applets/128) icons

2020-03-17 Thread Rocket Aaron 
rocka added a comment.


  Weird, I edited them in Inkscape …
  
  Aren't those icons used as default avatar in user manager?

REPOSITORY
  R266 Breeze Icons

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

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

D28094: [breeze-icons] fix shadow in some user (applets/128) icons

2020-03-17 Thread Rocket Aaron 
rocka updated this revision to Diff 77814.
rocka added a comment.


  remove svg metadata

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D28094?vs=77811=77814

BRANCH
  fix-user-icon-shadow (branched from master)

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

AFFECTED FILES
  icons-dark/applets/128/user-ada-lovelace.svg
  icons-dark/applets/128/user-leonardo-da-vinci.svg
  icons/applets/128/user-ada-lovelace.svg
  icons/applets/128/user-leonardo-da-vinci.svg

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

D28094: [breeze-icons] fix shadow in some user (applets/128) icons

2020-03-17 Thread Rocket Aaron 
rocka edited the summary of this revision.

REPOSITORY
  R266 Breeze Icons

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

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

D28094: [breeze-icons] fix shadow in some user (applets/128) icons

2020-03-17 Thread Rocket Aaron 
rocka edited the test plan for this revision.
rocka added a reviewer: VDG.

REPOSITORY
  R266 Breeze Icons

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

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

D28094: [breeze-icons] fix shadow in some user (applets/128) icons

2020-03-17 Thread Rocket Aaron 
rocka created this revision.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
rocka requested review of this revision.

REVISION SUMMARY
  When executing `kdialog --geticon`, some errors would shown:
  
$ kdialog --geticon
qt.svg: /usr/share/icons/breeze-dark/applets/128/user-ada-lovelace.svg:68: 
Could not resolve property: #linearGradient6624
qt.svg: /usr/share/icons/breeze-dark/applets/128/user-ada-lovelace.svg:68: 
Could not resolve property: #linearGradient6626
qt.svg: 
/usr/share/icons/breeze-dark/applets/128/user-leonardo-da-vinci.svg:79: Could 
not resolve property: #linearGradient6634
  
  ... And those 2 icons doesn't have correct shadow. This patch fixes it.

REPOSITORY
  R266 Breeze Icons

BRANCH
  fix-user-icon-shadow (branched from master)

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

AFFECTED FILES
  icons-dark/applets/128/user-ada-lovelace.svg
  icons-dark/applets/128/user-leonardo-da-vinci.svg
  icons/applets/128/user-ada-lovelace.svg
  icons/applets/128/user-leonardo-da-vinci.svg

To: rocka
Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D28093: [breeze-icons] add TeamViewer tray icons

2020-03-17 Thread Rocket Aaron 
rocka edited the test plan for this revision.

REPOSITORY
  R266 Breeze Icons

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

To: rocka
Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D28093: [breeze-icons] add TeamViewer tray icons

2020-03-17 Thread Rocket Aaron 
rocka added a reviewer: VDG.

REPOSITORY
  R266 Breeze Icons

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

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

D28093: [breeze-icons] add TeamViewer tray icons

2020-03-17 Thread Rocket Aaron 
rocka created this revision.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
rocka requested review of this revision.

REVISION SUMMARY
  Created TeamViewer tray icons following Breeze style.

REPOSITORY
  R266 Breeze Icons

BRANCH
  teamviewer-tray (branched from master)

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

AFFECTED FILES
  icons-dark/status/22/TeamViewer.svg
  icons/status/22/TeamViewer.svg

To: rocka
Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D27842: [breeze-icons] New 48px telegram icon

2020-03-04 Thread Rocket Aaron 
rocka added a comment.


  Yes, I submitted it via the Web UI.

REPOSITORY
  R266 Breeze Icons

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

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

D27842: [breeze-icons] New 48px telegram icon

2020-03-04 Thread Rocket Aaron 
rocka created this revision.
rocka added a reviewer: VDG.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
rocka requested review of this revision.

REVISION SUMMARY
  Since 48px telegram icon was removed in D27787 
, I made a new one following the official 
icon.

TEST PLAN
  256x256 PNG:
  
  F8149657: out.png 
  
  In panel with other icons:
  
  F8149659: image.png 

REPOSITORY
  R266 Breeze Icons

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

AFFECTED FILES
  icons-dark/apps/48/telegram.svg
  icons/apps/48/telegram.svg

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

D27617: [breeze-icons] Add telegram-desktop tray icons

2020-03-04 Thread Rocket Aaron 
rocka updated this revision to Diff 76946.
rocka added a comment.


  Fix attention dot color and remove accidentally added patch file

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27617?vs=76942=76946

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

AFFECTED FILES
  icons-dark/apps/22/telegram-attention-panel.svg
  icons-dark/apps/22/telegram-mute-panel.svg
  icons-dark/apps/22/telegram-panel.svg
  icons/apps/22/telegram-attention-panel.svg
  icons/apps/22/telegram-mute-panel.svg
  icons/apps/22/telegram-panel.svg

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

D27617: [breeze-icons] Add telegram-desktop tray icons

2020-03-04 Thread Rocket Aaron 
rocka updated this revision to Diff 76942.
rocka edited the test plan for this revision.
rocka added a comment.


  I've re-make the distinguishing part of the icons following the official one, 
it would be easy to recognize now. See the "Update" part in "Test Plan".
  
  Since the name for tray icons are `telegram-{,attention-,mute-}panel`, it 
would not affect the application icon elsewhere, whose name is 
`telegram-desktop`.

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27617?vs=76267=76942

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

AFFECTED FILES
  icons-dark/apps/22/telegram-attention-panel.svg
  icons-dark/apps/22/telegram-mute-panel.svg
  icons-dark/apps/22/telegram-panel.svg
  icons/apps/22/telegram-attention-panel.svg
  icons/apps/22/telegram-mute-panel.svg
  icons/apps/22/telegram-panel.svg
  telegram-panel-breeze.patch

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

D27787: Remove 48px telegram icons

2020-03-03 Thread Rocket Aaron 
rocka added a comment.


  Could we please not remove this icon, or at least keep it available as an 
alternative? It fits the general style so well.

REPOSITORY
  R266 Breeze Icons

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

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

D27617: [breeze-icons] Add telegram-desktop tray icons

2020-03-02 Thread Rocket Aaron 
rocka added a comment.


  Are we ready for some reviews?

REPOSITORY
  R266 Breeze Icons

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

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

D27617: [breeze-icons] Add telegram-desktop tray icons

2020-02-24 Thread Rocket Aaron 
rocka added a comment.


  What's the version of telegram-desktop? Would it still be blue if change 
workspace theme to breeze-dark? If it works fine in breeze-dark and telegram 
version is 1.9.12, it maybe a upstream bug 

 , and was fixed in the latest 1.9.14 release.

REPOSITORY
  R266 Breeze Icons

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

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

D27617: [breeze-icons] Add telegram-desktop tray icons

2020-02-24 Thread Rocket Aaron 
rocka created this revision.
rocka added a reviewer: VDG.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
rocka requested review of this revision.

REVISION SUMMARY
  Recent version of telegram-desktop added support for using system icon theme 
for tray icon. This patch adds some icons following breeze style.
  
  According to comments on GitHub 

 , pass environemnt variable `TDESKTOP_DISABLE_TRAY_COUNTER=1` to 
telegram-desktop also make it follows KDE color scheme.

TEST PLAN
  Before: F8128166: Screenshot_20200224_173312.png 

  
  After:
  
  | breeze | breeze-dark |
  | F8128129: Screenshot_20200224_172135.png 
  | F8128123: 
Screenshot_20200224_172102.png    
|

REPOSITORY
  R266 Breeze Icons

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

AFFECTED FILES
  icons-dark/apps/22/telegram-attention-panel.svg
  icons-dark/apps/22/telegram-mute-panel.svg
  icons-dark/apps/22/telegram-panel.svg
  icons/apps/22/telegram-attention-panel.svg
  icons/apps/22/telegram-mute-panel.svg
  icons/apps/22/telegram-panel.svg

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