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

Search logos #3

Merged
merged 8 commits into from
Jul 4, 2024
Merged

Search logos #3

merged 8 commits into from
Jul 4, 2024

Conversation

lucasferreiralimax
Copy link
Member

Add Search Logos Functionality and Improve Filters

New feature that allows users to search for logos within our application.
Additionally, improvements have been made to the existing filters by adjusting the text with trim and toLowerCase methods to avoid conflicts and ensure accurate search results.

New Features

  1. Search Logos: Users can now search for specific logos using a search input. This feature enhances user experience by making it easier to find logos quickly.
  2. Case-Insensitive Search: The search functionality now handles case differences by converting all text to lowercase.
  3. Trimmed Input: Extra spaces are removed from the input to ensure more precise filtering.

Changes

  • Added a new input field for searching logos.
  • Updated the filtering logic to use trim and toLowerCase methods.
  • Implemented signals in the search (searchSignal) and techsList to enhance reactivity and ensure accurate updates based on user input.

@lucasferreiralimax lucasferreiralimax self-assigned this Jul 4, 2024
@lucasferreiralimax lucasferreiralimax merged commit daed678 into master Jul 4, 2024
1 check passed
@lucasferreiralimax lucasferreiralimax deleted the search-logos branch July 4, 2024 11:03
Copy link

github-actions bot commented Jul 4, 2024

Visit the preview URL for this PR (updated for commit d924800):

https://angular-techs-logos--pr3-search-logos-3zy2nyz9.web.app

(expires Thu, 11 Jul 2024 11:04:37 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 1e6be52cb30c4b8fc438baf494ecc8ac65a6ec48

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