Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: header overlap with other content in smaller screens #3320

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

HASH-0021
Copy link
Contributor

Issue:

  • Header is overlapping with the below content for smaller screens.

Changes:

  • Header is changed from position:fixed to position:sticky. This allows header to have position as fixed when scrolling and relative when it isn't.
  • Added right margin to logo as the logo and navigation buttons are too close for certain screen widths.
  • Refactored the code of dark mode toggle button to use the existing code design language.
  • Adjusted spacing for header and main elements.

Copy link

🙏 Thanks for your pull request @HASH-0021, The team will now review and merge this request. In the mean time why not check out some of the other opensource projects available, contributions are greatly appreciated!

Some of the most popular are


PR Statistics

#️⃣ PR Number: Line Additions: 🗑️ Line Deletions:
3320 13 9
📑 Files Changed: Repo Stars: 🔱 Total Forks:
3 1112 2908

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