Skip to content

Conversation

@layananes
Copy link
Contributor

@layananes layananes commented Oct 18, 2025

Make navbar logo link to homepage for improved navigation

This change wraps the JargonsdevLogo component in a link (<a href="/">) so users can click the logo to return to the homepage.

Fixes #243

Summary

  • Wrapped the logo component with an anchor tag linking to /
  • Ensures the logo remains visually identical (size, drop shadow, layout)
  • Improves accessibility and follows standard web navigation conventions

Files Modified

  • src/components/navbar.astro

Testing Checklist

  • Logo is clickable and links to the homepage (/)
  • Styling and layout remain unchanged
  • Link works from all pages

Enhances user experience by providing an intuitive way to navigate back to the main page.

@vercel
Copy link

vercel bot commented Oct 18, 2025

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

Project Deployment Preview Comments Updated (UTC)
jargons-dev Ready Ready Preview Comment Oct 18, 2025 1:31pm

💡 Enable Vercel Agent with $100 free credit for automated AI reviews

@babblebey babblebey added the 🔴 wontmerge This pull request will not be merged for now label Oct 18, 2025
Copy link
Member

@babblebey babblebey left a comment

Choose a reason for hiding this comment

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

Hi @layananes

This PR intends to fix #200, the issue in this case hasn't been assigned to you, it is already assigned to someone and currently being worked on in PR #207, though it is now stale and work still needs to be done on it.

Here's what we'll do... We will check on the person working on the PR, if they do not complete the PR in another moment, the issue will be unassigned from them and will be opened again for you to work on... Then we can get back to this PR.

In the meantime, kindly look for other issues you can work on in the repo. 😉

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

Labels

🔴 wontmerge This pull request will not be merged for now

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants