Skip to content

Search Bar feature#47

Closed
Hrusikesh001 wants to merge 3 commits intoAvishkarPatil:mainfrom
Hrusikesh001:feature-3
Closed

Search Bar feature#47
Hrusikesh001 wants to merge 3 commits intoAvishkarPatil:mainfrom
Hrusikesh001:feature-3

Conversation

@Hrusikesh001
Copy link
Contributor

📝 Description

  • This PR adds a Search Bar feature to improve content discoverability within the platform.
  • Implemented a search bar at the top of the resources page.
  • Users can search for specific water conservation topics, articles, or resources.
  • Search works in real-time and dynamically filters results as the user types.
  • Added a fallback message when no results are found (“No results found for your search.”).
  • Styled the search bar with a clean, rounded design and integrated a search icon.

🔗 Related Issue

Fixes #8

🔄 Type of Change

✨ New feature (non-breaking change which adds functionality)
🎨 UI/UX improvement

🧪 Testing

  • I have performed a self-review of my code
  • Code has been tested locally
  • Verified search works across multiple datasets
  • No new warnings introduced

📱 Frontend Changes (if applicable)

  • Responsive design tested

  • Cross-browser compatibility checked (Chrome, Firefox, Edge)

  • Accessibility guidelines followed

🔧 Backend Changes (if applicable)

No Changes

📸 Screenshots/Demo

image

📋 Additional Notes

  • The search bar enhances navigation and improves overall user experience.
  • Future enhancement: Add highlighting of matched keywords in search results.

✅ Checklist

  • My code follows the project's coding standards
  • I have updated documentation where necessary
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works

@AvishkarPatil . I’ve added the Search Bar feature. Could you please review & merge and let me know if it looks good?

@github-actions
Copy link

Thanks for creating a PR for your Issue! ☺️

We'll review it as soon as possible.
In the meantime, please double-check the file changes and ensure that all commits are accurate.

If there are any unresolved review comments, feel free to resolve them. 🙌🏼

Copy link
Owner

@AvishkarPatil AvishkarPatil left a comment

Choose a reason for hiding this comment

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

Thanks for adding the global search feature! However, the navbar layout is completely broken.

Issues:

  • Navbar structure is messed up
  • Search bar integration broke existing layout
  • Mobile responsiveness likely affected
  • Navigation links may be overlapping

Please fix the navbar layout issues before we can merge this. The search functionality is great, but it shouldn't break the core navigation!

@Hrusikesh001
Copy link
Contributor Author

ok i will fix this

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.

Add a Search Functionality

3 participants