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

[OTHER] Restructure Header #2438

Closed
3 tasks done
Jay-Karia opened this issue Jul 29, 2024 · 3 comments · Fixed by #2434
Closed
3 tasks done

[OTHER] Restructure Header #2438

Jay-Karia opened this issue Jul 29, 2024 · 3 comments · Fixed by #2434

Comments

@Jay-Karia
Copy link
Contributor

What would you like to share or ask?

According to me, the header should be updated due to:

  1. Cluttered Social Icons
    Social icons are grouped with theme toggler, app links, and sponsor.

  2. Inappropriate position of Saved link.
    The right hand side of header belongs to external links.
    We can add the link beside the logo.

We can add sponsor button at the bottom of the sidebar.

Checklist

  • I have checked the existing issues
  • I have read the Contributing Guidelines
  • I am willing to work on this issue (optional)
Copy link

#2436, #2435, #2434, #2430, #2428, #2427 is already opened by you
As per contributing guidelines, you can only work on 1 issue at a time.

Copy link

Hello Jay-Karia!
Thank you for raising this issue! 😊 Your contribution is valuable to us! 😊

Please make sure to follow our Contributing Guidelines. 💪🏻

Please only work on an issue if you're assigned; otherwise, the PR will be automatically closed.
Our review team will carefully assess the issue and reach out to you soon! 😇
We appreciate your patience!

Copy link

github-actions bot commented Aug 8, 2024

Thank you for contributing to LinksHub! Please take a moment to rate this repo's DX on EddieHub's RepoRater and give it a star ⭐

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant