-
-
Notifications
You must be signed in to change notification settings - Fork 304
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
[IMP] mgmtsystem: Update icon #565
Conversation
How did you manage to create this V17 style icon? |
@bosd Our graphic designer made it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks awesome
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks awesome indeed.
Could you update also the svg
file?
Both png
and svg
need to be updated and kept in sync.
The svg
one is used to be able to easily maintain it in the future
@ivantodorovich Done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks ❤️ looks really good
I would check the dimensions of the png
image. 591 x 587
seems too large IMO.
I encourage you to check the dimensions in standard Odoo apps logos, and then use the same dimensions.
This PR has the |
/ocabot merge no-bump |
Hi @max3903. Your command failed:
Ocabot commands
More information
|
/ocabot merge bump |
Hi @max3903. Your command failed:
Ocabot commands
More information
|
/ocabot merge nobump |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at c3a0a4e. Thanks a lot for contributing to OCA. ❤️ |
@max3903