Skip to content

Normalize landing page vertical spacing#1524

Open
parkhojeong wants to merge 6 commits intoapache:mainfrom
parkhojeong:fix/community-page-mobile-tablet-spacing
Open

Normalize landing page vertical spacing#1524
parkhojeong wants to merge 6 commits intoapache:mainfrom
parkhojeong:fix/community-page-mobile-tablet-spacing

Conversation

@parkhojeong
Copy link
Copy Markdown
Contributor

@parkhojeong parkhojeong commented May 7, 2026

Previously, the vertical spacing across pages was inconsistent. This work normalizes it across the site.

Overview

Before After
image image
Before After
image image

spacing rule

  • top spacing: navbar(fixed) height + custom spacing per page(base-layout: 40px, others: custom per page)
  • bottom spacing: 40px(desktop)

checked list with screenshot

/ image
image
/community image
image
/community/resources/ image
image
/meetups/ image
image
/docs/ image
image
/use-cases/ image
image
/use-cases/etl_analytics/ image
image
/announcements/ image
image
/blog image
image
/blog/airflow-survey-2025/ image
image
/ecosystem/ image
image
/code-of-conduct/ image
image

cc @choo121600

…-spacing' into fix/community-page-mobile-tablet-spacing

# Conflicts:
#	landing-pages/site/assets/scss/_community-page.scss
@parkhojeong parkhojeong marked this pull request as ready for review May 7, 2026 08:21
@choo121600
Copy link
Copy Markdown
Member

If similar issues exist on other pages (event, use-case, ecosystem, etc.), could we fix them using a consistent style as well?

@parkhojeong
Copy link
Copy Markdown
Contributor Author

If similar issues exist on other pages (event, use-case, ecosystem, etc.), could we fix them using a consistent style as well?

Sure. I'll take a look

@parkhojeong parkhojeong changed the title Improve community page spacing on mobile and tablet Normalize landing page vertical spacing May 8, 2026
@parkhojeong
Copy link
Copy Markdown
Contributor Author

@choo121600
I checked the affected pages and normalized the landing page vertical spacing.

I added the checked page list with screenshots to the PR description. Could you take another look?

Copy link
Copy Markdown
Member

@choo121600 choo121600 left a comment

Choose a reason for hiding this comment

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

The layout structure of each page appears to be similar, but the styling approach is not consistent.

In some places, the page styles are modified, while in others the container styles are adjusted. This mixed approach could increase maintenance complexity over time.

If the layout structure is the same, it would be better to consolidate these into shared/common styles.

@parkhojeong
Copy link
Copy Markdown
Contributor Author

The layout structure of each page appears to be similar, but the styling approach is not consistent.

In some places, the page styles are modified, while in others the container styles are adjusted. This mixed approach could increase maintenance complexity over time.

If the layout structure is the same, it would be better to consolidate these into shared/common styles.

@choo121600

Thanks, that’s a good point. I agree

I have a question about the design guidelines. It’s a bit unclear what to base the spacing on. do you have any references, like a Figma mockup?

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.

2 participants