Re-design the sponsors page#118
Open
mrbiggred wants to merge 36 commits into
Open
Conversation
…sistent command execution
…firm plugin setup, and ensure privacy compliance
Based on a Claude Design mockup. Added cards for how to help and a nicer way to show the years. Also added sponsor boxes with move info on hover.
- Renamed "Support" section to "Contribute" - Enhanced support section with clearer descriptions - Updated CSS for lead paragraph and support cards - Included additional CSS files in mkdocs configuration
Contributor
There was a problem hiding this comment.
Pull request overview
Redesigns the Sponsors page to be data-driven (YAML-backed) and adds updated styling, with MkDocs Macros used to render current/past sponsors and a Stripe link for sponsorship.
Changes:
- Add
mkdocs-macros-pluginand configure MkDocs to loaddata/sponsors.ymlfor sponsor rendering. - Rebuild
docs/sponsors/index.mdto generate sponsor cards by year and add a Stripe sponsorship CTA. - Introduce dedicated Sponsors page CSS and update README/AGENTS guidance for maintaining sponsors and running MkDocs.
Reviewed changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| requirements.txt | Adds pinned transitive deps for mkdocs-macros-plugin. |
| requirements.in | Adds mkdocs-macros-plugin to the source dependency list. |
| README.md | Documents how to update sponsors via data/sponsors.yml and notes new structure/assets. |
| mkdocs.yml | Enables macros plugin YAML inclusion and adds sponsors stylesheet + watch data/. |
| docs/stylesheets/sponsors.css | Adds scoped styling for the redesigned sponsors page. |
| docs/stylesheets/extra.css | Updates site-wide h1 typography styling. |
| docs/sponsors/index.md | Replaces static content with macros-driven sponsor rendering + CTA. |
| data/sponsors.yml | Adds sponsor/donor data model with per-year references. |
| AGENTS.md | Updates contributor guidance to use Docker Compose for MkDocs commands. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Markdown inside the wdc-sponsors-page <div> doesn't render because md_in_html isn't enabled, so the warning showed as literal text. The fallback was dev-only and any bad sponsor ID is caught during local mkdocs serve. Addresses PR #118 review feedback.
Contributor
|
When I click the sponsors link I get the page, but missing some assets: Small change: |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The sponsor page has been re-designed and can now do the following:
Once this page is live, we can message previous donors who purchased tickets on Luma and see if they want to be added. We will keep the list of donors on a private spreadsheet and only add members that want to be public. They can be anonymous or have a nick-name listed.