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

[Integration logo AMD] - Logo is out of proportion #5309

Closed
sandramsc opened this issue Jan 20, 2024 · 5 comments
Closed

[Integration logo AMD] - Logo is out of proportion #5309

sandramsc opened this issue Jan 20, 2024 · 5 comments

Comments

@sandramsc
Copy link
Member

Link: 270+ Built-In Integrations
Section under "270+ Built-In Integrations"

Currently, the AMD integration logo looks out of proportion.

Desired state

Adjust the proportions of the logo to accurately reflect the original design.

Screenshots

Before

amd2

After

amdaft


Contributor Resources and Handbook

The layer5.io website uses Gatsby, React, and GitHub Pages. Site content is found under the master branch.

Join the Layer5 Community by submitting your community member form.

@sandramsc
Copy link
Member Author

CC: @iArchitSharma

@prabhjeevnijjar
Copy link
Contributor

Hey Sandra, I just opened the same issue #5308 a couple hours earlier

@nikhilkalburgi
Copy link
Contributor

I added object-fit: contain and this is the result:
image

If this is fine then I can raise a PR

@nikhilkalburgi
Copy link
Contributor

Also check this possible solution:

  alt={item.frontmatter.title}
- height = {70}
- width = {70}
+ height={(item.frontmatter.title == 'AMD GPU')? 'auto' : '70'}
+ width={(item.frontmatter.title == 'AMD GPU')? '95' : '70'}
  style={{
  filter:

Result:
image

Some icons are very small and also can be improved with above logic:
image

@sandramsc
Copy link
Member Author

Hey Sandra, I just opened the same issue #5308 a couple hours earlier

Hi @prabhjeevnijjar Thank you for this, closing issue as duplicate.

@sandramsc sandramsc reopened this Jan 21, 2024
@sandramsc sandramsc closed this as not planned Won't fix, can't repro, duplicate, stale Jan 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants