Skip to content

Conversation

@ivan-kalachikov
Copy link
Contributor

@ivan-kalachikov ivan-kalachikov commented Oct 16, 2025

@ivan-kalachikov ivan-kalachikov requested a review from a team as a code owner October 16, 2025 07:29
@ivan-kalachikov ivan-kalachikov requested review from Andrew-Orlov, goldenmaya and yuskithedeveloper and removed request for a team October 16, 2025 07:29
goldenmaya
goldenmaya previously approved these changes Oct 17, 2025
goldenmaya
goldenmaya previously approved these changes Oct 27, 2025
@sonarqubecloud
Copy link

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements a no-results search page feature by adding localization strings for empty search results and enhancing the search and category components to handle empty search states with improved user experience.

Key changes:

  • Added header_empty localization key across all language files for displaying empty search results messages
  • Modified search navigation to use router.replace when already on search page to prevent history stack buildup
  • Implemented empty search state detection and conditional UI rendering to hide filters and controls when no results are found

Reviewed Changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 2 comments.

File Description
locales/*.json Added header_empty translation key for empty search results message in 13 language files
client-app/shared/layout/components/search-bar/search-bar.vue Updated navigation logic to replace route when already on search page instead of pushing
client-app/shared/catalog/composables/useProducts.ts Added hasSelectedFilters function and computed property to check for active filters
client-app/shared/catalog/components/category.vue Implemented empty search state detection and conditional rendering of UI elements, added navigation logic for reset functionality

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

4 participants