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

fix: update Podman AI Lab logo #1774

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jeffmaury
Copy link
Contributor

Fixes #1660

What does this PR do?

Update Podman AI Lab logo

Screenshot / video of UI

image

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

Fixes containers#1660

Signed-off-by: Jeff MAURY <jmaury@redhat.com>
Copy link
Collaborator

@benoitf benoitf left a 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 ?

@jeffmaury
Copy link
Contributor Author

resolution is much more smaller, expected ?

@MariaLeonova can you clarify ?

@MariaLeonova
Copy link

resolution is much more smaller, expected ?

@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:

https://github.com/containers/podman-desktop-internal/blob/main/ai-lab/logo/new%20logo/svg/ai-lab-icon-dark-scheme.svg

@jeffmaury
Copy link
Contributor Author

We are using PNG format

Copy link
Collaborator

@benoitf benoitf left a 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

@MariaLeonova
Copy link

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.

@MariaLeonova
Copy link

Alright - these should be square, big and on a transparent background.
Dark is for dark scheme, light is for light.

ai-lab-icon-dark-big
ai-lab-icon-light-big

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.

Update AI Lab logo and icon with new version
3 participants