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

Simplify adopter information on the homepage #170

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

Conversation

travisn
Copy link
Member

@travisn travisn commented Jan 15, 2025

In the interest of equal representation of rook upstream adopters, remove the logos of the few companies and add a link to the full list of adopters. The company logos have anyway not been updated since the list was first created, and is not realistic to maintain.

@travisn travisn requested a review from jbw976 January 15, 2025 23:10
@travisn travisn mentioned this pull request Jan 15, 2025
In the interest of equal representation of rook upstream
adopters, remove the logos of the few companies
and add a link to the full list of adopters. The
company logos have anyway not been updated since
the list was first created, and is not realistic
to maintain.

Signed-off-by: Travis Nielsen <[email protected]>
@travisn travisn force-pushed the remove-adopter-logos branch from 14d1cc0 to 294c1f6 Compare January 15, 2025 23:15
Copy link
Member

@jbw976 jbw976 left a comment

Choose a reason for hiding this comment

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

Looks good from looking at the raw HTML - were you able to test the site with real rendering too? if so, feel free to share a screenshot 😉

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.

2 participants