-
Notifications
You must be signed in to change notification settings - Fork 31
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
fix: update Podman AI Lab logo #1774
base: main
Are you sure you want to change the base?
Conversation
Fixes containers#1660 Signed-off-by: Jeff MAURY <jmaury@redhat.com>
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.
resolution is much more smaller, expected ?
@MariaLeonova can you clarify ? |
@benoitf resolution of what is much smaller, please? I can make it any size that's needed and the svg files are available here. @jeffmaury it looks like the light version was used in the screenshot you provided, here's the dark scheme icon file: |
We are using PNG format |
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.
before the image was W: 621px | H: 621px
now: 100px/97px
so it's not a square anymore (I would expect to be 100x100 or 97x97) and the size has been reduced by 6 in width
It was 621x612 px for the icon on the left? That seems a bit much, but certainly can be done! I'll update it to be square. |
Fixes #1660
What does this PR do?
Update Podman AI Lab logo
Screenshot / video of UI
What issues does this PR fix or reference?
N/A
How to test this PR?
Launch Podman AI Lab and check the icon in the left bar