ndavis added a comment.

  Great. Now you just need to get rid of the invisible rectangle. You can do 
that quickly with `sed -i 's|<rect x="9.5367e-8" y="284.3" width="12.7" 
height="12.7" fill="none" opacity=".35"/>||' 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.

REPOSITORY
  R266 Breeze Icons

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

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

Reply via email to