Skip to content

refac: Move GithubLink from ThemeToggle to AppHeader#159

Merged
O2sa merged 1 commit into
O2sa:mainfrom
namahu:refac/move-githublink
Jun 18, 2026
Merged

refac: Move GithubLink from ThemeToggle to AppHeader#159
O2sa merged 1 commit into
O2sa:mainfrom
namahu:refac/move-githublink

Conversation

@namahu

@namahu namahu commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Fixes #158

This PR moves the compact GithubLink from ThemeToggle to AppHeader so the link is owned by the header layout, while keeping the header actions aligned with a small spacing adjustment.

Additionally, at the time of submission of this PR:

  • The referred issue is not blocked currently
  • All unittests passed after changes were made

@vercel

vercel Bot commented Jun 18, 2026

Copy link
Copy Markdown

@namahu is attempting to deploy a commit to the osama's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

Copy link
Copy Markdown

Thank you for the pull request! ✅

A maintainer will review this soon. Please be patient while we take a look. 🙌

@vercel

vercel Bot commented Jun 18, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dev-impact Ready Ready Preview, Comment Jun 18, 2026 7:59pm

@O2sa O2sa merged commit d4233bd into O2sa:main Jun 18, 2026
4 checks passed
@github-actions

Copy link
Copy Markdown

🎊 Welcome, @namahu! Your first contribution has been merged! 🚀

Thank you for helping improve the project! If you find this tool useful, please consider giving us a ⭐ star on GitHub—it helps more developers find our work and motivates us to keep improving!

@namahu namahu deleted the refac/move-githublink branch June 19, 2026 02:52
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.

refactor: Move GithubLink from ThemeToggle to AppHeader

2 participants