Conversation
dwdougherty
left a comment
There was a problem hiding this comment.
One broken link; otherwise, LGTM.
cmilesb
left a comment
There was a problem hiding this comment.
This is a partial review. I'd be happy to make some of these changes myself in a separate PR.
layouts/home.html
Outdated
|
|
||
| <!-- Redis Cloud --> | ||
| <div class="flex flex-col bg-redis-ink-900 border-2 border-redis-red-500 rounded-md overflow-hidden"> | ||
| <div class="px-4 py-2 border-b border-redis-pen-400"> |
There was a problem hiding this comment.
I think it would be useful to have these be buttons as well leading to the home page of each section. In my testing changing "div" to "a" seems to work, and adding the style to transition colors. Would be happy to make the changes myself.
layouts/home.html
Outdated
| </div> | ||
|
|
||
| <!-- Redis Cloud --> | ||
| <div class="flex flex-col bg-redis-ink-900 border-2 border-redis-red-500 rounded-md overflow-hidden"> |
There was a problem hiding this comment.
I don't mind the border and red text here too much, but I really don't like the fact that the background is the same color as the page background.
I'd recommend changing this color to match the software color, but then the Red color doesn't contrast well enough for the text. (Actually, as it stands, the text/background could be contrasted a little more - it seems to pass most of the tests but not all of them.) Happy to test colors myself here to see what works best. Maybe also a "Recommended" badge to make it stand out?
There was a problem hiding this comment.
Thank you so much Cameron! Yes, any changes you want to work on would be great.
|
The main thing I notice is that the full-width layout and the narrow mobile layout work OK, but in the "intermediate" width layout, the nav buttons at the left display on top of the content. However, this also happens with the current layout, so maybe we've decided it's not an issue? Anyway, I think the new layout looks pretty good and it crams more information into a smaller space. |
Co-authored-by: Cameron Bates <102550101+cmilesb@users.noreply.github.com>
Co-authored-by: Cameron Bates <102550101+cmilesb@users.noreply.github.com>
Co-authored-by: Cameron Bates <102550101+cmilesb@users.noreply.github.com>
Doc home update edits
🛡️ Jit Security Scan Results✅ No security findings were detected in this PR
Security scan by Jit
|
- Move AI chatbot search box to align with 'Redis for AI and search' heading - Center H1 'Welcome to Redis Docs' heading - Update 'Redis Cloud' link to 'Redis Flex' with new target URL - Remove AI chatbot search from 'Explore the documentation' section - Adjust spacing and alignment throughout
|
The thing with the nav bar appearing over the content boxes still seems to be there but otherwise I'd say it's looking really good :-) |
| </a> | ||
| </div> | ||
| <p class="text-sm text-gray-300 text-center"> | ||
| Search the <a href='{{ relURL "commands" }}' class="text-redis-yellow-500 hover:underline">complete command reference</a> and the <a href='{{ relURL "integrate" }}' class="text-redis-yellow-500 hover:underline">integrated libraries and tools</a> |
There was a problem hiding this comment.
Inconsistent link styles break staging navigation
Medium Severity
The new content sections mix two different link styles: {{ relURL "commands" }} and {{ relURL "integrate" }} on line 62 vs ./develop/, ./operate/rc/, etc. everywhere else. On staging (served under a subpath like /docs/staging/doc-home-update/), relURL resolves from the site root while ./ resolves relative to the current page. This means the "complete command reference" and "integrated libraries and tools" links navigate to a different path hierarchy than every other link on the page. The PR reviewer flagged this exact concern about ./ vs direct links for staging consistency.
Additional Locations (1)
Added a link to the Redis Knowledge Base with a brief description.
Updated the support article description and removed the community link.
- Added fourth tile for Redis for Kubernetes in product continuum section - Updated grid layout to support 4 tiles (responsive: 1/2/4 columns) - Changed Kubernetes tile buttons to Overview and Quick start - Increased font size for links in 'Choose your next step' section (xs -> sm) - Increased font size for titles in 'Explore the documentation' section (xs -> sm) - Updated description text to mention Kubernetes option
- Replace RAG box with Agent Builder (develop/ai/agent-builder/) - Replace Streaming box with Vector Database (develop/get-started/vector-database/) - Update descriptions to match new content focus
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
|
|
||
| </div> | ||
| </div> | ||
| </section> |
There was a problem hiding this comment.
Partial docs-section.html is now completely unused
Low Severity
This commit removes all references to the docs-section.html partial (previously called ~12 times to render the home page content sections). A grep confirms docs-section.html is no longer referenced anywhere in the codebase, making layouts/partials/docs-section.html dead code.


Here is a new draft of the docs home page in response to feedback that the current one is overwhelming and to help make the main options more clear at the top of the page. Any and all feedback welcomed and no rush on this one https://redis.io/docs/staging/doc-home-update/
Note
Low Risk
Static template/layout changes only; risk is limited to navigation/UX regressions or broken links on the homepage.
Overview
Redesigns the docs home page to be less overwhelming by replacing the prior dense
docs-section.html-driven “Get Started”/“Deploy & Operate” content with a simpler, intent-based landing.The new layout emphasizes two primary CTAs (“Develop with Redis Open Source” and “Run Redis in production”), adds an “Redis for AI and search” section with an embedded
/chatsearch box, use-case links, and an AI stack visual, and introduces new “product continuum”, “choose your next step”, and “explore the documentation” link sections. Minor styling tweaks include applyingfont-monomore broadly and shortening the AI search placeholder text.Written by Cursor Bugbot for commit 6b1bb64. This will update automatically on new commits. Configure here.