Skip to content

feat(ui): redesign homepage with new visual elements #234

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Apr 16, 2025

Conversation

phiro56
Copy link
Contributor

@phiro56 phiro56 commented Apr 4, 2025

Description:

Redesign homepage with new visual elements

  • Add hero section with gradient background and new hero image
  • Implement new documentation cards layout
  • Enhance "What's New" section styling

Notes:
New header and footer design will be updated in a separate PR

screencapture-127-0-0-1-1111-2025-04-04-09_33_29

- Add hero section with gradient background and new hero image
- Implement new documentation cards layout
- Enhance "What's New" section styling

Signed-off-by: Daniel Phillips <[email protected]>
@phiro56 phiro56 force-pushed the feat-ui-redesign-home branch from 92e37fa to ec77598 Compare April 4, 2025 17:07
Copy link

@makubo-aws makubo-aws left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@madolson madolson requested a review from Copilot April 15, 2025 00:10
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

Files not reviewed (1)
  • sass/_valkey.scss: Language not supported
Comments suppressed due to low confidence (3)

templates/index.html:3

  • The block name has been changed from 'main_content' to 'content'; please ensure that the default.html base template defines the corresponding block to maintain consistent layout rendering.
{%- block content -%}

templates/index.html:87

  • Reassigning the global variable 'major' inside the loop may unintentionally overwrite the original value used for displaying the most recent release. Consider using a separate local variable for older versions to avoid potential version display issues.
{% set_global major = older_major %}

content/_index.md:59

  • Verify that the '+++' marker correctly terminates the front matter. If this marker is not intended for front matter delimitation, it might lead to parsing issues in some markdown processors.
+++

@makubo-aws
Copy link

@stockholmux can you take a look?

phiro56 and others added 4 commits April 16, 2025 15:37
Co-authored-by: Madelyn Olson <[email protected]>
Signed-off-by: Daniel Phillips <[email protected]>
Co-authored-by: Madelyn Olson <[email protected]>
Signed-off-by: Daniel Phillips <[email protected]>
Co-authored-by: Madelyn Olson <[email protected]>
Signed-off-by: Daniel Phillips <[email protected]>
Co-authored-by: Madelyn Olson <[email protected]>
Signed-off-by: Daniel Phillips <[email protected]>
Signed-off-by: Madelyn Olson <[email protected]>
@madolson madolson merged commit d1da33a into valkey-io:main Apr 16, 2025
1 check passed
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