Skip to content

fix: remove duplicate homepage top spacing#300

Open
adityakumar-cs24 wants to merge 1 commit intoEswaramuthu:mainfrom
adityakumar-cs24:fix/hero-section-spacing
Open

fix: remove duplicate homepage top spacing#300
adityakumar-cs24 wants to merge 1 commit intoEswaramuthu:mainfrom
adityakumar-cs24:fix/hero-section-spacing

Conversation

@adityakumar-cs24
Copy link
Copy Markdown

Description

Fixes UI spacing issue caused by duplicate offset handling between header and main content.

Changes made

  • Removed extra margin-top from .main-content
  • Kept .header spacer for fixed navbar
  • Preserved scrolling and layout behavior

Result

  • Hero section now aligns properly under navbar
  • Cleaner and more balanced UI

Type of change

  • Bug fix (non-breaking change)

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 17, 2026

@adityakumar-cs24 is attempting to deploy a commit to the 007's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link
Copy Markdown

Thanks for creating a PR for your Issue! ☺️

We'll review it as soon as possible.
In the meantime, please double-check the file changes and ensure that all commits are accurate.

If there are any unresolved review comments, feel free to resolve them. 🙌🏼

@entelligence-ai-pr-reviews
Copy link
Copy Markdown

LGTM 👍 No issues found.

@entelligence-ai-pr-reviews
Copy link
Copy Markdown


Confidence Score: 5/5 - Safe to Merge

Safe to merge — this PR removes duplicate top spacing on the homepage, a focused and low-risk CSS/layout fix with no logic, runtime, or security implications. The change is scoped to a single file and the review found zero issues across all changed files. No pre-existing unresolved concerns are associated with this PR.

Key Findings:

  • The PR addresses a specific visual defect (duplicate top spacing on the homepage), which is a straightforward, well-scoped fix with no risk of side effects beyond layout.
  • Full review coverage was achieved (1/1 changed files reviewed) and no issues — critical, significant, or minor — were identified by the automated analysis.
  • No logic, data handling, or security-sensitive code paths are touched; the change is purely presentational.

@adityakumar-cs24
Copy link
Copy Markdown
Author

This fix removes duplicate spacing caused by both header and main content offsets. Let me know if any changes are needed.

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.

1 participant