D27926: [breeze-icons] Add flameshot tray icons

2020-03-08 Thread Ariel AxionL 
axionl marked an inline comment as done.
axionl added a comment.


  In D27926#624240 , @ngraham wrote:
  
  > Nice!
  >
  > Do you think you could also make 16px versions?
  
  
  Sorry, maybe not. The 16px space is too small to get the correct angle to 
convert 1px thick strokes to paths on Inkscape.

REPOSITORY
  R266 Breeze Icons

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

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

D27926: [breeze-icons] Add flameshot tray icons

2020-03-08 Thread Ariel AxionL 
axionl updated this revision to Diff 77223.
axionl added a comment.


  Update flameshot tray icon.

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27926?vs=77197=77223

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

AFFECTED FILES
  icons-dark/status/22/flameshot-tray.svg
  icons/status/22/flameshot-tray.svg

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

D27926: [breeze-icons] Add flameshot tray icons

2020-03-07 Thread Ariel AxionL 
axionl retitled this revision from "Add flameshot tray icons" to 
"[breeze-icons] Add flameshot tray icons".

REPOSITORY
  R266 Breeze Icons

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

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

D27926: Add flameshot tray icons

2020-03-07 Thread Ariel AxionL 
axionl created this revision.
axionl added a reviewer: VDG.
axionl added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
axionl requested review of this revision.

REVISION SUMMARY
  Add the system tray icon for flameshot 
. This patch adds some icons 
following the breeze style.

TEST PLAN
  Before:
  F8165736: 图片.png 
  
  After:
  F8165738: 图片.png 
  
  F8165739: 图片.png 

REPOSITORY
  R266 Breeze Icons

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

AFFECTED FILES
  icons-dark/status/22/flameshot-tray.svg
  icons/status/22/flameshot-tray.svg

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

D19324: Add code-oss icon

2019-02-28 Thread Ariel AxionL 
axionl updated this revision to Diff 52826.
axionl added a comment.


  - Setting my username to pass the automated audit.

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D19324?vs=52786=52826

BRANCH
  code-oss (branched from master)

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

AFFECTED FILES
  icons-dark/apps/48/code-oss.svg
  icons-dark/apps/48/code.svg
  icons/apps/48/code-oss.svg
  icons/apps/48/code.svg

To: axionl, #vdg, ndavis
Cc: ndavis, rooty, ngraham, kde-frameworks-devel, michaelh, bruns

D19324: Add code-oss icon

2019-02-27 Thread Ariel AxionL 
axionl edited the test plan for this revision.

REPOSITORY
  R266 Breeze Icons

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

To: axionl, #vdg
Cc: ndavis, rooty, ngraham, kde-frameworks-devel, michaelh, bruns

D19324: Add code-oss icon

2019-02-27 Thread Ariel AxionL 
axionl added a comment.


  In D19324#421178 , @ndavis wrote:
  
  > Great. Now you just need to get rid of the invisible rectangle. You can do 
that quickly with `sed -i 's|||' code.svg`. After you've done that, 
I can accept and land this.
  >
  > BTW, you're meant to put the comment for the diff update at the top, but 
it's not a huge deal. That's why you keep getting this text in all of your 
update comments:
  >
  > - Merge branch 'master' into code-oss
  >  
  >   # Updating D19324: Add code-oss icon
  >   #
  >   # Enter a brief description of the changes included in this update.
  >   # The first line is used as subject, next lines as comment.
  >   #
  >   # If you intended to create a new revision, use:
  >   #  $ arc diff --create
  >
  >
  > The stuff to the left of the `#` characters is ignored if you put the 
comment at the top.
  
  
  This is my first time to contribute plasma/kde project, I feel sorry for my 
mistakes and thanks for your help.

REPOSITORY
  R266 Breeze Icons

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

To: axionl, #vdg
Cc: ndavis, rooty, ngraham, kde-frameworks-devel, michaelh, bruns

D19324: Add code-oss icon

2019-02-27 Thread Ariel AxionL 
axionl updated this revision to Diff 52786.
axionl added a comment.


  - Merge branch 'master' into code-oss
  
  Remove the invisible rectangle.

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D19324?vs=52721=52786

BRANCH
  code-oss (branched from master)

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

AFFECTED FILES
  icons-dark/apps/48/code-oss.svg
  icons-dark/apps/48/code.svg
  icons/apps/48/code-oss.svg
  icons/apps/48/code.svg

To: axionl, #vdg
Cc: ndavis, rooty, ngraham, kde-frameworks-devel, michaelh, bruns

D19324: Add code-oss icon

2019-02-27 Thread Ariel AxionL 
axionl updated this revision to Diff 52719.
axionl added a comment.


  - Merge branch 'master' into code-oss
  
  1. Updating D19324 : Add code-oss icon #
  2. Enter a brief description of the changes included in this update.
  3. The first line is used as subject, next lines as comment. #
  4. If you intended to create a new revision, use:
  5. $ arc diff --create
  
  Replace the green accent and optimize the svg

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D19324?vs=52696=52719

BRANCH
  code-oss (branched from master)

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

AFFECTED FILES
  icons-dark/apps/48/code-oss.svg
  icons-dark/apps/48/code.svg
  icons/apps/48/code-oss.svg
  icons/apps/48/code.svg

To: axionl, #vdg
Cc: ndavis, rooty, ngraham, kde-frameworks-devel, michaelh, bruns

D19324: Add code-oss icon

2019-02-27 Thread Ariel AxionL 
axionl updated this revision to Diff 52721.
axionl added a comment.


  - Merge branch 'master' into code-oss
  
  1. Updating D19324 : Add code-oss icon #
  2. Enter a brief description of the changes included in this update.
  3. The first line is used as subject, next lines as comment. #
  4. If you intended to create a new revision, use:
  5. $ arc diff --create
  
  Optimize svg

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D19324?vs=52719=52721

BRANCH
  code-oss (branched from master)

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

AFFECTED FILES
  icons-dark/apps/48/code-oss.svg
  icons-dark/apps/48/code.svg
  icons/apps/48/code-oss.svg
  icons/apps/48/code.svg

To: axionl, #vdg
Cc: ndavis, rooty, ngraham, kde-frameworks-devel, michaelh, bruns

D19324: Add code-oss icon

2019-02-27 Thread Ariel AxionL 
axionl added a comment.


  In D19324#420976 , @ndavis wrote:
  
  > I see you've kept the green on the light theme version instead of using the 
same icon for both versions. Is there a reason for this or do you just prefer 
it?
  >
  > This looks just about ready to land on the master branch, there are just a 
few more things you need to do.
  
  
  Just I like it as a modern design style and also avoiding some problems for 
the similarity of official icon, but if you mind that I can remove it.

REPOSITORY
  R266 Breeze Icons

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

To: axionl, #vdg
Cc: ndavis, rooty, ngraham, kde-frameworks-devel, michaelh, bruns

D19324: Add code-oss icon

2019-02-26 Thread Ariel AxionL 
axionl edited the test plan for this revision.

REPOSITORY
  R266 Breeze Icons

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

To: axionl, #vdg
Cc: ndavis, rooty, ngraham, kde-frameworks-devel, michaelh, bruns

D19324: Add code-oss icon

2019-02-26 Thread Ariel AxionL 
axionl updated this revision to Diff 52696.
axionl added a comment.


  - Merge branch 'master' into code-oss
  
  1. Updating D19324 : Add code-oss icon #
  2. Enter a brief description of the changes included in this update.
  3. The first line is used as subject, next lines as comment. #
  4. If you intended to create a new revision, use:
  5. $ arc diff --create
  
  Try to fix the layer and shadows.

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D19324?vs=52588=52696

BRANCH
  code-oss (branched from master)

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

AFFECTED FILES
  icons-dark/apps/48/code-oss.svg
  icons-dark/apps/48/code.svg
  icons/apps/48/code-oss.svg
  icons/apps/48/code.svg

To: axionl, #vdg
Cc: ndavis, rooty, ngraham, kde-frameworks-devel, michaelh, bruns

D19380: # Enter a commit message. # # Changes: # # icons-dark/apps/48/code.svg # icons/apps/48/code.svg

2019-02-26 Thread Ariel AxionL 
axionl abandoned this revision.

REPOSITORY
  R266 Breeze Icons

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

To: axionl
Cc: bruns, kde-frameworks-devel, michaelh, ngraham

D19380: # Enter a commit message. # # Changes: # # icons-dark/apps/48/code.svg # icons/apps/48/code.svg

2019-02-26 Thread Ariel AxionL 
axionl added a comment.


  I make a mistake to create a new revision, please delete this one.

REPOSITORY
  R266 Breeze Icons

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

To: axionl
Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D19380: # Enter a commit message. # # Changes: # # icons-dark/apps/48/code.svg # icons/apps/48/code.svg

2019-02-26 Thread Ariel AxionL 
axionl created this revision.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
axionl requested review of this revision.

REVISION SUMMARY
  Tyr to fix the layer and the shadow

REPOSITORY
  R266 Breeze Icons

BRANCH
  code-oss

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

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

To: axionl
Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D19324: Add code-oss icon

2019-02-26 Thread Ariel AxionL 
axionl updated this revision to Diff 52588.
axionl added a comment.


  - Merge branch 'master' into code-oss
  
  1. Updating D19324 : Add code-oss icon #
  2. Enter a brief description of the changes included in this update.
  3. The first line is used as subject, next lines as comment. #
  4. If you intended to create a new revision, use:
  5. $ arc diff --create
  
  fixed color details.

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D19324?vs=52587=52588

BRANCH
  code-oss (branched from master)

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

AFFECTED FILES
  icons-dark/apps/48/code-oss.svg
  icons-dark/apps/48/code.svg
  icons/apps/48/code-oss.svg
  icons/apps/48/code.svg

To: axionl, #vdg
Cc: ndavis, rooty, ngraham, kde-frameworks-devel, michaelh, bruns

D19324: Add code-oss icon

2019-02-26 Thread Ariel AxionL 
axionl edited the test plan for this revision.

REPOSITORY
  R266 Breeze Icons

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

To: axionl, #vdg
Cc: ndavis, rooty, ngraham, kde-frameworks-devel, michaelh, bruns

D19324: Add code-oss icon

2019-02-26 Thread Ariel AxionL 
axionl edited the test plan for this revision.

REPOSITORY
  R266 Breeze Icons

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

To: axionl, #vdg
Cc: ndavis, rooty, ngraham, kde-frameworks-devel, michaelh, bruns

D19324: Add code-oss icon

2019-02-26 Thread Ariel AxionL 
axionl updated this revision to Diff 52587.
axionl added a comment.


  - Merge branch 'master' into code-oss
  
  1. Updating D19324 : Add code-oss icon #
  2. Enter a brief description of the changes included in this update.
  3. The first line is used as subject, next lines as comment. #
  4. If you intended to create a new revision, use:
  5. $ arc diff --create
  
  Redraw by inkscape.

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D19324?vs=52581=52587

BRANCH
  code-oss (branched from master)

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

AFFECTED FILES
  icons-dark/apps/48/code-oss.svg
  icons-dark/apps/48/code.svg
  icons/apps/48/code-oss.svg
  icons/apps/48/code.svg

To: axionl, #vdg
Cc: ndavis, rooty, ngraham, kde-frameworks-devel, michaelh, bruns

D19324: Add code-oss icon

2019-02-25 Thread Ariel AxionL 
axionl added a comment.


  In D19324#419767 , @ndavis wrote:
  
  > In D19324#419766 , @axionl wrote:
  >
  > > I use the Gravit Designer (https://designer.io/) to design this icon, not 
the inkscape, but I think I can redraw it by inkscape.
  >
  >
  > Please do. KDE software is only made with free and open source software. If 
you need help with Inkscape, let us know. We would be glad to help.
  >
  > > For the filename, because I'm using archlinux and the icon name is 
`code-oss.svg` on `/usr/share/pixmaps`.
  >
  > In that case, run `ln -sr code-oss.svg code.svg` in the same directory as 
`code-oss.svg` to made a symlink so that the icon works for Arch Linux and any 
other distro.
  
  
  Ok, I am done as soon as possible.

REPOSITORY
  R266 Breeze Icons

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

To: axionl, #vdg
Cc: ndavis, rooty, ngraham, kde-frameworks-devel, michaelh, bruns

D19324: Add code-oss icon

2019-02-25 Thread Ariel AxionL 
axionl added a comment.


  In D19324#419751 , @ndavis wrote:
  
  > Weird. In Inkscape, the logo looks like raster graphics, but in Dolphin and 
Gwenview it looks like vector graphics. Something is seriously screwed up in 
this SVG. Did you use Inkscape? Can you explain what process you used to make 
this icon?
  >  F6636063: Screenshot_20190226_020059.png 

  >  F6636069: Screenshot_20190226_020524.png 

  >
  > In D19324#419700 , @ngraham 
wrote:
  >
  > > > `code-oss-dark.svg`
  > >
  > > Is this the correct name?
  >
  >
  > I don't think so. The Code-OSS flatpak and the VS Code RPM use `code.png`, 
so I think the name needs to be `code.svg` for light and dark Breeze icons. It 
would be a good idea to test if using icon themes actually works though, 
because `code.png` installs to 
`/usr/share/code/resources/app/resources/linux/code.png` and 
`/usr/share/pixmaps/code.png` instead of hicolor.
  
  
  I use the Gravit Designer (https://designer.io/) to design this icon, not the 
inkscape, but I think I can redraw it by inkscape.
  
  For the filename, because I'm using archlinux and the icon name is 
`code-oss.svg` on `/usr/share/pixmaps`.

REPOSITORY
  R266 Breeze Icons

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

To: axionl, #vdg
Cc: ndavis, rooty, ngraham, kde-frameworks-devel, michaelh, bruns

D19324: Add code-oss icon

2019-02-25 Thread Ariel AxionL 
axionl updated this revision to Diff 52581.
axionl added a comment.


  - Merge branch 'master' into code-oss

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D19324?vs=52571=52581

BRANCH
  code-oss (branched from master)

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

AFFECTED FILES
  icons-dark/apps/48/code-oss.svg
  icons/apps/48/code-oss.svg

To: axionl, #vdg
Cc: ndavis, rooty, ngraham, kde-frameworks-devel, michaelh, bruns

D19324: Add code-oss icon

2019-02-25 Thread Ariel AxionL 
axionl added a comment.


  In D19324#419702 , @rooty wrote:
  
  > There are commands/aliases here for how to take quick screenshots 
https://invent.kde.org/ndavis/hig-kde-org/wikis/Icon-Workflow
  >
  > I ran montage-breeze and I got F6635927: new.png 

  >  which looks really nice.
  >
  > However, is there some reason that it looks different from a Dolphin 
generated thumbnail? F6635929: image.png 
  >
  > EDIT: And yes the dark version should have the same name as the regular 
version (code-oss.svg).
  
  
  I have the same problem but this icon working well and can be display normaly 
on my firebrowser.

REPOSITORY
  R266 Breeze Icons

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

To: axionl, #vdg
Cc: rooty, ngraham, kde-frameworks-devel, michaelh, bruns

D19324: Add code-oss icon

2019-02-25 Thread Ariel AxionL 
axionl created this revision.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
axionl requested review of this revision.

REVISION SUMMARY
  Add the code-oss icons follow the breeze and breeze-dark style

REPOSITORY
  R266 Breeze Icons

BRANCH
  code-oss (branched from master)

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

AFFECTED FILES
  icons-dark/apps/48/code-oss-dark.svg
  icons/apps/48/code-oss.svg

To: axionl
Cc: kde-frameworks-devel, michaelh, ngraham, bruns