home: make taking the salary survey the call to action - #2785
Draft
ericholscher wants to merge 1 commit into
Draft
ericholscher wants to merge 1 commit into
ericholscher wants to merge 1 commit into
Conversation
The card pointed at the results, with taking the survey as a secondary link in the copy. The survey only works if people fill it in, so make that the card, and move the results into the copy. Which one shows is driven by the closing date in conf.py, the way the announcement banner works, so the card falls back to the results once the 2026 survey closes on 31 January 2027. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015JdLb6hXkD9Vym4eABf8pg
Documentation build overview
|
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
2 participants
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 salary survey card from #2780 led with the results and buried taking the survey as a secondary link. That's backwards while submissions are open — the results only exist because people fill it in, and the ask is the time-sensitive half.
The card now asks, and the results move into the copy. Which version shows is driven by the closing date in
conf.py, following the announcement banner pattern, so it reverts to pointing at the results after the 2026 survey closes on 31 January 2027 rather than inviting people to a closed form. Next year, updating the year and date in one place switches it back.Also renames the sponsorship card to "Become a sponsor", matching the footer button. It was the one title long enough to wrap to two lines, which made that row sit taller than the first.
Worth a look
index.rst. A config-only change doesn't invalidate it on an incremental build — Read the Docs builds clean, so in practice it turns over on the first build after the closing date rather than on the date itself.conf.pyalongside the date, so the two can't drift apart in the copy.Generated by Claude Code
📚 Documentation preview 📚: https://writethedocs-www--2785.org.readthedocs.build/