Docs: add sign up and log in CTA to footer#12841
Open
ericholscher wants to merge 4 commits intomainfrom
Open
Conversation
Documentation build overview
Show files changed (2 files in total): 📝 2 modified | ➕ 0 added | ➖ 0 deleted
|
Documentation build overview
Show files changed (14 files in total): 📝 14 modified | ➕ 0 added | ➖ 0 deleted
|
Member
Author
|
Updated to surface both sites directly in the footer. Each one now gets its own card with a small header, a one-sentence description of who it's for, and direct sign up / log in links — no modals, no JS, the choice between Community and Business is made right in the footer.
Generated by Claude Code |
Member
Author
|
@agjohnson This should be more mergable as is. |
Add a call-to-action banner in the docs footer with sign up and log in buttons that open modals for choosing between Community and Business platforms, matching the website pattern. Also fix templates_path in conf.py to use an absolute path so the multiproject plugin srcdir override does not break template resolution. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Drop the sign-up and log-in modals (and their associated markup, JavaScript, and CSS) in favor of two anchor buttons that link directly to the Community signup and login pages. This keeps the footer CTA unobtrusive and removes ~200 lines of bespoke modal styling that we'd otherwise have to maintain in the docs theme.
Show both readthedocs.org (Community) and readthedocs.com (Business) side-by-side in the docs footer, each with a small header, a one-line description, and direct sign up / log in links. Cards stack on narrow viewports.
Use the cyan-teal #0993af (and matching #00839d hover / #00778e focus) from the about.readthedocs.com Semantic UI theme for the primary Sign up buttons, plus the matching #1a1e22 dark surface for the CTA background. Keeps the footer CTA visually consistent with the destination signup/login pages on the marketing site.
ericholscher
force-pushed
the
docs/footer-cta
branch
from
ce18f4e to
a024264
Compare
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.
Summary
readthedocs.org) and Business (readthedocs.com)#0993afprimary,#1a1e22dark surface)templates_pathinconf.pyto use an absolute path so the multiproject plugin'ssrcdiroverride doesn't break template resolutionTest plan
PROJECT=user make html) and verify the CTA appears at the bottom of every pageapp.readthedocs.orgapp.readthedocs.comScreenshots will be attached as a follow-up comment.