Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add m/monit icon #847

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from
Open

add m/monit icon #847

wants to merge 4 commits into from

Conversation

victorclaessen
Copy link

This time to the dev branch (replaces #845).

@rchiileea if you could be more specific in what is wrong with the svg, that would be much appreciated!

add monit icon
Now it should be 20x20 with 2x2 offset, so there is a margin of 2 around all sides.
now monit is 20x20
@victorclaessen
Copy link
Author

Thanks to @rchiileea 's comment in #845 I now think I understand, so I updated this PR.
I resized the logo to 20x20 and gave it an offset of 2x2 so that it now has a margin of 2 around all sides in the 24x24 box.

I hope this is more to your liking!

@elax46 elax46 requested a review from rchiileea December 3, 2024 22:02
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.

1 participant