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 :-) |
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
|
|
||
| </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.
- Changed 'Vector Sets' to 'Vector sets' - Changed 'Agent Builder' to 'Agent builder' - Changed 'Vector Database' to 'Vector database'
- Build AI apps box: Replaced with comprehensive client library links - Added Redis vector library (Python), redis-py, node-redis, Jedis, Lettuce, NRedisStack, go-redis, redis-rs, Predis, hiredis - Removed description text to fit more links - Run AI in production box: Updated integrations and tools - Moved Operate Redis to top of list - Changed 'Semantic cache' to 'LangCache' - Replaced 'Security & Compliance' with 'Agent skills' - Added Redis MCP, LangChain, Amazon Bedrock, Spring Data - Kept description text - Get Redis tools box: Enhanced description - Updated description to mention JSON and vector queries in Redis Sandbox - Added Agent skills link
…ions - Added new 'Data integration & observability' section with 6 tiles: - RIOT, Redis Data Integration, Prometheus & Grafana - Datadog, Dynatrace, New Relic - Build AI apps box: Removed all parentheticals from client library names - Cleaner presentation (e.g., 'Python' instead of 'Python (redis-py)') - Updated .NET to C#/.NET - Run AI in production box: Removed Agent skills and Redis MCP - Now has 6 links: Operate Redis, Flex, LangCache, LangChain, Amazon Bedrock, Spring Data - Get Redis tools box: Added Redis MCP and updated description - Changed description to mention 'CLI, GUI, VS Code, and MCP tools' - Added Redis MCP link to the list
- Add icons to Redis for AI and search boxes - Add icons to product boxes (Open Source, Software, Kubernetes, Cloud) - Update stack diagram with new colors and badge-style Redis logo - Change product boxes to clickable cards with yellow hover effect - Remove horizontal borders between sections - Left-justify all sections for consistent alignment - Reduce vertical spacing between sections for more compact layout - Update section headings to use Space Grotesk font - Change Vector search box to AI development with updated link and description - Update middle layer styling with Hyper-10 background and Hyper-Base border - Change arrow colors from yellow to grey in stack diagram
| <h1 class="text-2xl md:text-3xl font-sans font-medium text-white text-center">Welcome to Redis Docs</h1> | ||
| <div class="flex flex-col items-center gap-4"> | ||
| <div class="flex flex-col md:flex-row gap-4 justify-center"> | ||
| <a href="./develop/" class="px-8 py-4 bg-redis-red-10 border border-redis-red-500 hover:bg-redis-red-500 text-white font-medium rounded-md transition-colors text-center"> |
There was a problem hiding this comment.
Undefined Tailwind color redis-red-10 produces no background
High Severity
The class bg-redis-red-10 doesn't exist in the Tailwind config. The redis-red palette in tailwind.config.js only defines shades 500, 600, and 700. Tailwind won't generate any CSS for this class, so the "Develop with Redis Open Source" CTA button and the Redis highlight layer in the AI stack visual will have no background color — they'll appear transparent instead of the intended tinted red.
Additional Locations (2)
…o next steps boxes, and add Contribute section
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 3 potential issues.
There are 5 total unresolved issues (including 2 from previous reviews).
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
| </a> | ||
|
|
||
| <!-- PHP --> | ||
| <a href="./develop/clients/phpredis/" class="hover:opacity-70 transition-opacity"> |
There was a problem hiding this comment.
PHP icon links to nonexistent path
High Severity
The PHP language icon links to ./develop/clients/phpredis/ but the actual content path is content/develop/clients/php/ and no alias for phpredis exists. This results in a 404. The "Build fast apps" section on the same page correctly uses ./develop/clients/php/.
Additional Locations (1)
| </a> | ||
|
|
||
| <!-- Rust --> | ||
| <a href="./develop/clients/redis-rs/" class="hover:opacity-70 transition-opacity"> |
There was a problem hiding this comment.
Rust icon links to nonexistent path
High Severity
The Rust language icon links to ./develop/clients/redis-rs/ but the actual content path is content/develop/clients/rust/ and no alias for redis-rs exists. This results in a 404. The "Build fast apps" section on the same page correctly uses ./develop/clients/rust/.
Additional Locations (1)
| </a> | ||
|
|
||
| <!-- Ruby --> | ||
| <a href="./develop/clients/redis-rb/" class="hover:opacity-70 transition-opacity"> |
There was a problem hiding this comment.
Ruby icon links to nonexistent documentation page
High Severity
The Ruby language icon links to ./develop/clients/redis-rb/ but no such content page exists. Ruby is only listed as a community-supported client in the clients index page without its own documentation directory or alias, so this link results in a 404.
… Redis in production button
- Change all box borders from redis-pen-400 to redis-pen-700 for consistency - Update base font from font-mono to font-sans (Space Grotesk) throughout - Change chatbot placeholder to 'Ask our chatbot to explain' - Move AI Agent Resources to Contribute section with icon - Add Tutorials box to Additional Resources section - Update Contribute section grid to accommodate 4 boxes


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
Medium Risk
Large rewrite of the docs homepage markup and navigation content, with many new sections/links and inline SVGs; risk is primarily UX/regression (layout, responsiveness, broken/incorrect links), not backend behavior.
Overview
Redesigns
layouts/home.htmlto make the Docs landing page more task-oriented: a simplified welcome header (smaller typography + chatbot prompt) and prominent CTAs for developing with open source vs running in production.Replaces the previous long list of
docs-section.htmlcards with new curated sections: language quick links (icon grid), AI/search use-case entry points (plus a small “stack” visual), a product continuum grid (OSS → Cloud), intent-based “Choose your next step” routing, plus dedicated blocks for data integration/observability, external learning resources, and contribution/feedback.Written by Cursor Bugbot for commit 6c72554. This will update automatically on new commits. Configure here.