Skip to content

Conversation

@yeskunall
Copy link

@yeskunall yeskunall commented Jun 19, 2025

Description

Some company logos on the home page had incorrect names assigned to the alt and title attributes.

ChatGPT appears twice right now, so I also dropped the redundant instance.

Related Issues

N/A

How can it be tested?

Hover over the company logos to check the title is set correctly.

Check list before submitting

  • If this PR added new pages, they have been added to the sitemap and their metadata is clear and optimized (N/A)
  • I have performed a self-review of my code (no debugs, no commented code, good naming, etc.)
  • I updated the documentation if necessary (N/A)
  • This PR is wrote in a clear language and correctly labeled

Some company logos on the home page had incorrect names assigned to the
`alt` and `title` attributes.
@vercel
Copy link

vercel bot commented Jun 19, 2025

@yeskunall is attempting to deploy a commit to the Manifest Team on Vercel.

A member of the Team first needs to authorize it.


<div class="stack-grid-item is-hidden-desktop">
<img
src="./assets/images/stack/logo-openai.svg"
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: this asset be renamed to logo-chatgpt given that it actually represents the logo for ChatGPT and not OpenAI, but I didn’t make that change before getting you guys’ approval.

@yeskunall
Copy link
Author

PS: the company logos’ layout looks weird on mobile and tablet:

Mobile CleanShot 2025-06-19 at 15 15 58@2x
Tablet CleanShot 2025-06-19 at 15 16 53@2x

Do you want that fixed as well?

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