Skip to content

[WIP] Add Vinegar icon#517

Open
YCaTGameRT wants to merge 3 commits into
linuxmint:masterfrom
YCaTGameRT:master
Open

[WIP] Add Vinegar icon#517
YCaTGameRT wants to merge 3 commits into
linuxmint:masterfrom
YCaTGameRT:master

Conversation

@YCaTGameRT

Copy link
Copy Markdown

@Hokiper

Hokiper commented Sep 8, 2025

Copy link
Copy Markdown
Contributor

Did you draw the symbol manually? They provide an official vector file in that repository you sent: https://github.com/vinegarhq/vinegar/blob/master/data/icons/vinegar.svg

Also there are some continuity issues here (mainly with the shadows).
image

If you were trying to create a monochrome version, it would be better to do it with Inkscape's boolean operations like so:
cinnamon-2025-09-08T114728-0300.webm

@Hokiper

Hokiper commented Sep 8, 2025

Copy link
Copy Markdown
Contributor

(If you do it that way, remember to delete these nodes after the operation)
image

@YCaTGameRT

Copy link
Copy Markdown
Author

@Hokiper, thanks, i updated the icons and now they should look better

@Hokiper

Hokiper commented Sep 10, 2025

Copy link
Copy Markdown
Contributor

There is now an artifact here:
image

This can be avoided with the method I showed earlier in this screen recording:
cinnamon-2025-09-08T114728-0300.webm

Also the shadows are now way below the logo, instead of the 1 pixel displacement that you used previously and that is common in this theme.

@YCaTGameRT

Copy link
Copy Markdown
Author

I got rid of the artifacts, reduced the shadow, and made the wine glass more red

@clefebvre clefebvre changed the title Add Vinegar icon [WIP] Add Vinegar icon Nov 24, 2025
@e9x

e9x commented Dec 11, 2025

Copy link
Copy Markdown

#407 should be dealt with before this goes anywhere

We need to set up a mint-y-icons-games or something for this.

Is merging this the right direction for the repository?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants