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

Implement Logo Hover Effect #6086

Open
Tharanishwaran opened this issue Jan 26, 2025 · 5 comments
Open

Implement Logo Hover Effect #6086

Tharanishwaran opened this issue Jan 26, 2025 · 5 comments
Labels
discussion Input from everyone is helpful to drive this forward enhancement New feature or request ux

Comments

@Tharanishwaran
Copy link
Contributor

Desired feature or idea:

  • Implement a hover effect on the logos displayed on the OpenTelemetry.io website. When a user hovers over a logo, it should change to display the original trademark colors of that brand.

  • This would provide a more authentic and visually engaging experience for users. Attached are example images demonstrating the desired effect.

Current Behavior :

Screencast.from.26-01-25.11.30.22.PM.IST.webm

Desired Behavior :

Screencast.from.26-01-25.11.33.43.PM.IST.webm
@Tharanishwaran
Copy link
Contributor Author

can i work on this issue ?

@svrnm
Copy link
Member

svrnm commented Jan 27, 2025

@Tharanishwaran can you share some code (e.g. in a draft PR) how this would look like? On the one hand I like that feature, on the other hand I want to avoid that we introduce a lot of javascript/CSS overhead for just accomplishing this. I would especially appreciate if this would be a pure CSS-solution, which should be possible with modern features.

@open-telemetry/docs-approvers WDYT?

@svrnm svrnm added enhancement New feature or request ux discussion Input from everyone is helpful to drive this forward labels Jan 27, 2025
@Tharanishwaran
Copy link
Contributor Author

Thank you for the feedback @svrnm! I agree that a pure CSS solution would be ideal to minimize overhead. Here's how I'm thinking of implementing this:

We could use CSS filters to create the grayscale effect by default, then remove them on hover to reveal the original colors. Something like:

.logo-grid img {
  filter: grayscale(100%);
  transition: filter 0.3s ease;
}

.logo-grid img:hover {
  filter: grayscale(0%);
}

@svrnm
Copy link
Member

svrnm commented Jan 30, 2025

We could use CSS filters to create the grayscale effect by default, then remove them on hover to reveal the original colors. Something like:

The logos are from fontawesome and from what I see, they do not have color by default, so that would mean we need to replace them first with colored versions, so it's unfortunately not as easy as that, especially for colorful logos like the slack one

@chalin
Copy link
Contributor

chalin commented Jan 30, 2025

I'd (strongly) vote to stick with the Docsy default for footer icons, that is, to use tooltips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion Input from everyone is helpful to drive this forward enhancement New feature or request ux
Projects
Status: No status
Development

No branches or pull requests

3 participants