Skip to content

Doc home update edits#2785

Open
cmilesb wants to merge 2 commits intodoc-home-updatefrom
doc-home-update-edits
Open

Doc home update edits#2785
cmilesb wants to merge 2 commits intodoc-home-updatefrom
doc-home-update-edits

Conversation

@cmilesb
Copy link
Contributor

@cmilesb cmilesb commented Feb 12, 2026

These are the edits I thought about making to #2763 - let me know what you think.

https://redis.io/docs/staging/doc-home-update-edits/

@cmilesb cmilesb requested review from a team and mich-elle-luna February 12, 2026 15:30
@cmilesb cmilesb changed the base branch from main to doc-home-update February 12, 2026 15:30
Copy link
Contributor

@andy-stark-redis andy-stark-redis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One minor question but otherwise LGTM.

<!-- Redis Software -->
<div class="flex flex-col bg-redis-pen-800 border border-redis-pen-400 rounded-md overflow-hidden">
<div class="px-4 py-2 border-b border-redis-pen-400">
<a class="px-4 py-2 border-b border-redis-pen-400 hover:bg-redis-pen-700 transition-colors" href="./operate/rs/">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The background colour here is the same as the Redis Cloud box to the right, but different to Redis Open Source and all the other boxes on the page. Presumably there's no need to "lightly" emphasise RS here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Michelle said she wanted to emphasize RC & ROS slightly, and I just didn't like the fact that the original RC card blended in with the background. We could change all of the boxes to the ROS color, but that would make it stand out a little less. It's up to @mich-elle-luna in the end.

Copy link
Collaborator

@mich-elle-luna mich-elle-luna left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you Cameron!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants