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

feat: supporter page and updated home page #7552

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ovflowd
Copy link
Member

@ovflowd ovflowd commented Mar 14, 2025

This PR updates the Home Page and adds a new Supporter page with dynamic Supporter Icon rendering on Home Page and a tiered-Supported Logo List.

The Supporter page is added to the about/supporters section. This is a draft PR and contains fixtured/mocked supporter data.

CTAs

  • @mhdawson provide a list of current Node.js supporters
    • Do the outreach to companies that are supporters and get SVG logos and icons of said supporters
    • Revise the design and content of the Supporters page and Home Page
  • @ovflowd design tweaks as needed (like colours, spacing, positioning, etc)
  • Update copies and links on Home Page as needed

Screenshots

image image

Copy link

vercel bot commented Mar 14, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
nodejs-org ✅ Ready (Inspect) Visit Preview Mar 14, 2025 5:23pm

Copy link
Contributor

Note

Your Pull Request seems to be updating Translations of the Node.js Website.

Whilst we appreciate your intent; Any Translation update should be done through our Crowdin Project.
We recommend giving a read on our Translation Guidelines.

Thank you!

Copy link
Contributor

github-actions bot commented Mar 14, 2025

Lighthouse Results

URL Performance Accessibility Best Practices SEO Report
/en 🟢 100 🟢 100 🟢 100 🟢 91 🔗
/en/about 🟢 100 🟢 100 🟢 100 🟢 91 🔗
/en/about/previous-releases 🟢 99 🟢 100 🟢 100 🟢 92 🔗
/en/download 🟢 90 🟢 100 🟢 100 🟢 91 🔗
/en/blog 🟢 100 🟢 100 🟢 96 🟢 92 🔗

Copy link
Contributor

Unit Test Coverage Report

Lines Statements Branches Functions
Coverage: 90%
88.75% (742/836) 76.1% (242/318) 87.65% (142/162)

Unit Test Report

Tests Skipped Failures Errors Time
182 0 💤 0 ❌ 0 🔥 5.653s ⏱️

@bjohansebas
Copy link
Member

image
I would see this part as promoting one company over another, what do you think?

@ovflowd
Copy link
Member Author

ovflowd commented Mar 14, 2025

image

I would see this part as promoting one company over another, what do you think?

That is exactly the intent. But all companies will have their share. Apologies, but please avoid commenting if you are unfamiliar with the scope of these changes. I recommend giving a read within the docs on nodejs/admin repo :) -- or ask @mhdawson

But yes, companies with bigger thresholds should appear more, but all will appear here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants