Skip to content

Refresh the Apache Polaris homepage - #5279

Open
flyrain wants to merge 8 commits into
apache:mainfrom
flyrain:yf/refresh-homepage
Open

Refresh the Apache Polaris homepage#5279
flyrain wants to merge 8 commits into
apache:mainfrom
flyrain:yf/refresh-homepage

Conversation

@flyrain

@flyrain flyrain commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Why

The existing front page combines an oversized welcome message with a separate image that repeats the product identity. This change creates a clearer first impression, improves navigation to the most important resources, and presents the project consistently across desktop and mobile sizes.

Screenshot

image

Checklist

  • 🛡️ Don't disclose security issues! (contact security@apache.org)
  • 🔗 Clearly explained why the changes are needed, or linked related issues
  • 🧪 Added/updated tests with good coverage, or manually tested (and explained how)
  • 💡 Added comments for complex logic — no complex logic was introduced
  • 🧾 Updated CHANGELOG.md — not needed for a website-only presentation change
  • 📚 Updated documentation in site/content/in-dev/unreleased — not needed for this homepage-only change

@github-project-automation github-project-automation Bot moved this to PRs In Progress in Basic Kanban Board Aug 12, 2026
@flyrain
flyrain marked this pull request as ready for review August 12, 2026 06:04
Copilot AI lite review requested due to automatic review settings August 12, 2026 06:04

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR refreshes the Apache Polaris website homepage by moving from the previous blocks-based composition to a custom landing layout with a hero section, clearer navigation paths, and updated site metadata aligned to the “data and AI assets” tagline.

Changes:

  • Replaced the homepage content rendering with a dedicated hero/about/community landing layout.
  • Added homepage-specific SCSS for full-bleed hero styling and responsive behavior down to mobile widths.
  • Updated the homepage front matter description and introduced an official white brandmark SVG asset.

Reviewed changes

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

File Description
site/static/img/logos/polaris-brandmark-white.svg Adds the official Polaris brandmark asset for the refreshed hero.
site/layouts/home/list.html Implements the new homepage structure (hero, about, community links) and primary navigation CTAs.
site/content/_index.adoc Updates homepage metadata description and removes prior blocks-based homepage body content.
site/assets/scss/_styles_project.scss Adds homepage-specific styling for the new landing layout, including responsive rules.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread site/layouts/home/list.html Outdated
Comment thread site/assets/scss/_styles_project.scss
Comment thread site/assets/scss/_styles_project.scss Outdated
@flyrain
flyrain requested a review from dimas-b August 12, 2026 18:28
@flyrain
flyrain requested a review from sungwy August 12, 2026 22:26
adnanhemani
adnanhemani previously approved these changes Aug 13, 2026

@adnanhemani adnanhemani left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@github-project-automation github-project-automation Bot moved this from PRs In Progress to Ready to merge in Basic Kanban Board Aug 13, 2026
@MonkeyCanCode

MonkeyCanCode commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

On my 13.3-inch screen size and here is the comparison between current and new:

current:
image
image

new:
image
image

NITS: I think the white section feels a little out of place. Could we keep it in the same dark theme as the rest of the page. However, for the actual doc contents, we are using white theme (lack of dark mode). So I am fine with it as well. Perhaps wait for more feedbacks?

@dimas-b dimas-b left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

+1 to the top of the page changes.

The white section for "What is Apache Polaris" does look a bit awkward. I'd prefer this page to stay uniform in style top-to-bottom.

I'm not sure the "What is..." message is necessary. Why not simply describe what Polaris is without a question?

The old "welcome" section is much nicer IMHO:

Old:

Image

New:

Image

In general, let's keep this open until early September to allow more people to participate when they return from vacations.

@dimas-b dimas-b Aug 14, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why do we need a new image for this? The old site also has a similar image 🤔

@flyrain flyrain Aug 14, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The old image has words embedded. We could NOT use it.

I'm not sure the "What is..." message is necessary. Why not simply describe what Polaris is without a question?

I'm OK with either way. I can remove it if you feel strongly about it.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Removed the "what is ..." in the new commit

Comment thread site/content/_index.adoc
@flyrain

flyrain commented Aug 14, 2026

Copy link
Copy Markdown
Contributor Author

The old "welcome" section is much nicer IMHO:

using the old style now. Here is a screenshot:
Screenshot 2026-08-14 at 4 15 32 PM

@flyrain

flyrain commented Aug 14, 2026

Copy link
Copy Markdown
Contributor Author

Thanks for the review, @adnanhemani , @MonkeyCanCode and @dimas-b. Addressed all comments. PTAL.

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.

5 participants