Skip to content

fix: make navbar logo clickable and accessible#78

Open
bhavyagupta27 wants to merge 1 commit into
asheesh109:mainfrom
bhavyagupta27:main
Open

fix: make navbar logo clickable and accessible#78
bhavyagupta27 wants to merge 1 commit into
asheesh109:mainfrom
bhavyagupta27:main

Conversation

@bhavyagupta27

@bhavyagupta27 bhavyagupta27 commented Jun 5, 2026

Copy link
Copy Markdown

Summary

  • Brief description of changes

Related issue

  • Fixes: #ISSUE

Implementation notes

  • What changed and why

Checklist

  • Tests added/updated
  • Documentation updated
  • CHANGELOG updated (if applicable)

Screenshots (if applicable)

Summary

Provide a short description of the change and the motivation.

Related issues

  • Closes #

Type of change

  • Bug fix
  • New feature
  • Documentation update
  • Refactor

Checklist

  • Tests added/updated
  • Lint and type checks pass
  • Docs updated (if applicable)
  • PR linked to an issue (if applicable)

How to test

Provide steps to reproduce or verify the changes locally.

Screenshots (if applicable)


Mention reviewers and any special notes for maintainers.

📝 Description

Brief description of what this PR does.

🔗 Related Issues

Closes #(issue number)

Related to: #(issue number)

🎯 Type of Change

  • 🐛 Bug fix (non-breaking)
  • ✨ New feature (non-breaking)
  • 🔄 Breaking change
  • 📖 Documentation update
  • 🔧 Configuration change
  • ♻️ Refactoring

✅ Testing

What testing was done?

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing completed
  • Cross-browser tested (if UI change)
  • Mobile tested (if UI change)

Test Coverage

  • New code has tests
  • Existing tests pass
  • Coverage maintained or improved

How to Test

# Commands to test this PR
npm run test
npm run dev

📸 Screenshots (if applicable)

Before

After

🔍 Code Review Checklist

  • Code follows project style
  • No console logs (except errors)
  • No commented-out code
  • TypeScript types are complete
  • Error handling is proper
  • Security considerations reviewed
  • Performance implications considered

📝 Documentation

  • Updated relevant documentation
  • Updated README if needed
  • Updated API docs if applicable
  • Added code comments for complex logic
  • Updated CHANGELOG.md

🚀 Deployment

  • This is production-ready
  • No database migrations needed
  • No environment variables added
  • No breaking changes

📋 Additional Notes

Any additional context or information that reviewers should know?


Reviewers Please Note

Important points:

  • This PR fixes [specific issue]
  • Changes affect [specific areas]
  • Special attention needed for [specific code]

🎫 PR Checklist

  • All CI checks pass
  • No conflicts with base branch
  • Branch is up-to-date with main
  • Commit messages are clear
  • No unrelated changes

Thank you for contributing to KisanAI! 🌾

@vercel

vercel Bot commented Jun 5, 2026

Copy link
Copy Markdown

@bhavyagupta27 is attempting to deploy a commit to the ashishparab03-gmailcom's projects Team on Vercel.

A member of the Team first needs to authorize it.

@bhavyagupta27

Copy link
Copy Markdown
Author

Hi @asheesh109,

This PR makes the navbar logo clickable and adds an accessibility aria-label for homepage navigation.

Issue addressed: #9

Please let me know if any changes are needed. Thank you for your time and review.

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