Skip to content

Conversation

@2400030343SameerReddy
Copy link

@2400030343SameerReddy 2400030343SameerReddy commented Oct 22, 2025

This PR fixes an issue with inconsistent spacing above section headings.
It introduces additional spacing above headings to improve visual hierarchy and readability throughout the site.

Specifically:

Added top margin above headings (e.g., h2, h3, etc.) to ensure consistent vertical spacing.

Updated styles based on feedback from maintainers.

This change improves the overall layout and readability of content sections on the website.

Fixes #5648

Notes for Reviewers

Please verify that the spacing appears consistent across different pages.

Confirm that no additional unwanted gaps appear before other elements.

Signed commits

Yes, I signed my commits.

✅ Tip:
If your PR doesn’t directly close an issue, replace
Fixes #5648 → with just a short note like

“This PR improves visual spacing consistency above headings.”

@Rajesh-Nagarajan-11
Copy link
Member

PR Description needed

@2400030343SameerReddy 2400030343SameerReddy changed the title Add spacing above heading Add Sameer reddy to the contributors list Oct 23, 2025
@2400030343SameerReddy
Copy link
Author

@Rajesh-Nagarajan-11 i have made the changes in the description could you please accept my pull request.

@Rajesh-Nagarajan-11
Copy link
Member

Inappropriate pr title

@2400030343SameerReddy 2400030343SameerReddy changed the title Add Sameer reddy to the contributors list Add Spacing Above The Heading Oct 23, 2025
@2400030343SameerReddy
Copy link
Author

@Rajesh-Nagarajan-11 sir i have changed the title of the pr could you please accept it

Copy link
Member

@Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 left a comment

Choose a reason for hiding this comment

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

DCO Failed ⚠️
To add your Signed-off-by line to every commit in this branch:

  1. Ensure you have a local copy of your branch by checking out the pull request locally via command line.
  2. In your local branch, run: git rebase HEAD~2 --signoff
  3. Force push your changes to overwrite the branch: git push --force-with-lease origin add-spacing-above-heading

@Rajesh-Nagarajan-11
Copy link
Member

Builds and checks failed ⚠️

@Rajesh-Nagarajan-11
Copy link
Member

Thank you for your contribution!
Let's discuss this during the website call today at 5:30 PM IST | 7 AM CT
Add it as an agenda item to the meeting minutes, if you would 😄

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.

Add spacing above the heading

2 participants