Skip to content

Edit: Made Header and Sidebar a sticky component - #84

Open
TheInfernitex wants to merge 2 commits into
Ritikverma323:mainfrom
TheInfernitex:sticky-layout-fix
Open

Edit: Made Header and Sidebar a sticky component#84
TheInfernitex wants to merge 2 commits into
Ritikverma323:mainfrom
TheInfernitex:sticky-layout-fix

Conversation

@TheInfernitex

Copy link
Copy Markdown

Pull Request Template

Description

This PR implements simple UI/UX improvements by fixing the header at the top and making the sidebar sticky for persistent visibility, using explicit padding on the body to prevent all content obstruction

Fixes #74

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Other

Checklist

  • My code follows the repository style guidelines
  • I have performed a self-review of my changes
  • I have added necessary documentation (if applicable)

@vercel

vercel Bot commented Oct 10, 2025

Copy link
Copy Markdown

@TheInfernitex is attempting to deploy a commit to the RitikVerma's projects Team on Vercel.

A member of the Team first needs to authorize it.

@Ritikverma323

Copy link
Copy Markdown
Owner

@TheInfernitex, I have checked your PR Header is fixed but sidebar is not sticky on all pages. It is sticky on homepage but on all other pages it is not sticky.

@TheInfernitex

TheInfernitex commented Oct 15, 2025

Copy link
Copy Markdown
Author

Hi @Ritikverma323, thanks for the review! You are correct.

This is the intended functionality of position: sticky. It only activates when the content in the main container is tall enough to scroll. On the shorter pages, there's no scrollbar, so the the sidebar scrolls with the content. The sticky effect will work as intended once those pages have more content.

Did you want to have the sidebar to remain at the top, even after the main content has finished? Let me know your thoughts.

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.

[UI/UX] : Header and sidebar as a sticky component

2 participants