Thanks! The new icon looks good to me. However it looks like an xmlerrors file was accidentally added to this patch. Can you remove that?

Looks good. Since it's a proprietary program just check if they have any restrictions about having to use their icons. And yep, the xmlerrors file should be removed.

I think that Sublime Merge use Sublime Text license. It is free for use, too.

I think the cyan color is too bright and saturated. It fits the original branding well, but stands out from other Breeze icons.

I would suggest using a darker, more blue than cyan color.

Maybe this color will be better?

Ooh, I like it!

I don't that version is actually in the patch though. @otavva can you update this patch to have that version of the icon? Then we can get it landed for you.

In D20063#439465, @ngraham wrote:

I don't that version is actually in the patch though. @otavva can you update this patch to have that version of the icon? Then we can get it landed for you.

No, it's there, but for some reason I see myself as the author of the commit instead of @otavva

Oh wait no, it's not there yet, but it still looks pretty good.

However, I did notice this on a closer inspection of the shapes:

@otavva, could you make the top shape more like how it is in the official logo? You don't need to do the rounded corners, but that middle section is off.

https://www.sublimemerge.com/images/logo.svg

In D20063#439575, @ndavis wrote:

but for some reason I see myself as the author of the commit instead of @otavva

That's how it looks when you test a patch by someone who didn't use arc to produce it: you see your own authorship info instead of theirs. Sucks, I know...

Updated it.
@ndavis Done.

@ndavis
Should I to put it here? Sorry for stupid question, but I don't understand you fully. And what is Arcanist tool? I'm a first time here.

In D20063#440460, @otavva wrote:

@ndavis
Should I to put it here?

Yeah just put it in a comment.

And what is Arcanist tool? I'm a first time here.

https://community.kde.org/Infrastructure/Phabricator#Using_Arcanist_to_post_patches

Thanks so much @otavva! If you've found this process enjoyable and rewarding, there are plenty more icons needed! :)

See https://bugs.kde.org/buglist.cgi?bug_status=UNCONFIRMED&bug_status=CONFIRMED&bug_status=ASSIGNED&bug_status=REOPENED&component=Icons&list_id=1604229&product=Breeze&query_format=advanced