@chryw thanks for this awesome PR!
There appears to be a conflict for the continue.svg. Can you please resolve that conflict by taking your new version of that file and ignore what is on master? I can also do this if you are having issues.

Did you build vscode out of source and check if icons seem to fit in light and dark theme? I can also do this.

As for the build errors feel free to ignore those.

@isidorn
I resolved that conflict.

No I didn't build. I only sparse-checkouted that one folder and aligned the icon files. I've been using the same light/dark theme color map for all icons that I've made for vscode. I agree it's a good idea to actually look at these icons in UI. I haven't set up my computer to build vscode yet. Could you help me do a visual pass on your computer if it's convenient?

@chryw thanks for resolving the conflict. I will merge this in and verify all is good by building vscode.
Just for the future here are nice steps on how to build vscode on your machine https://github.com/Microsoft/vscode/wiki/How-to-Contribute#build-and-run

Thanks again for polishing all our icons, really appreciate your work 🍹

@isidorn Thanks! I'll set up my machine so I can help with more icon issues :-)

@chryw ok I will just comment here what I think does not work with the new icons:

• drag.svg - it is too small compared to the other debug toolbar icons
  

• breakpoint red is too bright in dark theme and is drawing too much attention, particularly visible on conditional breakpoints
  

• there are no current arrow both for light and dark theme (it is supposed to be on the top stack frame in the editor glyph margin when you start debugging). So all "current something .svg" seem to be broken

• there is no focussed arrow both for light and dark theme. So all "stack frame something.svg" seem to be broken

• Now that we have changed the settings wrench in debug should we also change the one on the bottom of the activity bar to be consistent?
  

• The debug bug is huge in the activity bar and I am afraid it will bite me
  

I like the new sexy remove breakpoint icon however I am not sure if we should use this icon also for the remove all watch expressions. Since the circles represent breakpoints for me. Though this is up for debate

Everything else seems to be good. Great work

Also let me know if you do not have time to fix these today and I will temporarly revert your commit so we do not get a bunch of bugs tomorrow from insider users.

Since i am going to bed I will temporarly revert the commit on master until we address the issues I mentioned above. Thanks

@isidorn I'm fixing these today:

• drag icons size: I plan to change the css: width: 16px, height: 16px;
• bright red color in dark theme: I plan to have dark theme point to the same svg file as light theme: breakpoint.svg (and more). Let's not introduce another red color to icon palette just for these.
• add stack frame arrow icons
• debug icon size in activity bar: I plan to add css: -webkit-mask-size: 28px;. svg icons should be responsive and not have size hard-coded. What do you think?

The "remove-all.svg" was meant to be used just for removing breakpoints. For clearing watch expressions, there is "clear-repl.svg" icon. Did you mean we should use that one?

@isidorn np. I'll send another PR tomorrow. Thanks! :-)

@chryw great! All your points make sense, feel free to hack around in .css
As for the remove-all for the watch. I think I would prefer sticking with our previous icon. Especially since it is consistent with the colapse all icon which is next to it

Got it. I'll grab a multi-box clear icon then.