Skip to content

Feature: add breadcrumb to search page#55

Merged
andres15alvarez merged 10 commits into
devfrom
fix/ECOM/navigation-and-search-experience-improvements
May 5, 2025
Merged

Feature: add breadcrumb to search page#55
andres15alvarez merged 10 commits into
devfrom
fix/ECOM/navigation-and-search-experience-improvements

Conversation

@rikardodlpj88
Copy link
Copy Markdown
Contributor

@rikardodlpj88 rikardodlpj88 commented Apr 26, 2025

Pull Request: Enhance Category Carousel, Search Suggestions & Breadcrumbs
Summary of Changes
Category Carousel

  • Added a new CategoryCarousel component to display product categories in a horizontal carousel.
  • Enabled filtering by category: clicking on a carousel item now navigates to the search page with the selected category applied.
    Responsive Search Suggestions
  • Refactored SearchSuggestions to be fully responsive, switching between 1–4 column layouts based on viewport size.
  • Ensured the suggestions dropdown adapts to mobile and desktop breakpoints seamlessly.
    Improved SearchBar Filtering
  • Fixed product filtering and search behavior in SearchBar when selecting a category from the dropdown.
  • Ensured that both the search icon and suggestion clicks clear the input and close the dropdown, preventing stale suggestions.
    Breadcrumbs
  • Added breadcrumb navigation to the Home and Search pages for better context and navigation hierarchy.
  • Corrected the breadcrumb route on the Product Detail page to use the proper URL parameters.

@rikardodlpj88 rikardodlpj88 added the feature New feature or request label Apr 26, 2025
@rikardodlpj88 rikardodlpj88 self-assigned this Apr 26, 2025
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 26, 2025

Deploy Preview for pharmatech ready!

Name Link
🔨 Latest commit 3613c56
🔍 Latest deploy log https://app.netlify.com/sites/pharmatech/deploys/6818dbc83fbfac000839f493
😎 Deploy Preview https://deploy-preview-55--pharmatech.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@rikardodlpj88 rikardodlpj88 changed the title WIP: add breadcrumb to search page Feature: add breadcrumb to search page Apr 27, 2025
@rikardodlpj88 rikardodlpj88 marked this pull request as ready for review April 27, 2025 02:09
@FranciscoMCR
Copy link
Copy Markdown
Contributor

Hay un comportamiento que estamos echando en falta, y es el siguiente: Cuando un usuario va a la vista detalle de una categoría el breadcrumb está bien que diga el nombre de la categoría, sin embargo, hay yo agregar cualquier otro filtro (una categoría mas por ejemplo) ese breadcrumb debería cambiar a algo como "Filtro personalizado", ya luego si voy hacia el detalle de un producto si recupero el breadcrumb de la categoría y debería poder volver a esa categoría filtrada, esto porque no vamos a trabajar con persistencia de filtros por ahora

@rikardodlpj88

@FranciscoMCR
Copy link
Copy Markdown
Contributor

Aún no me es posible ver las sugerencias de búsqueda desde mobile @rikardodlpj88

@FranciscoMCR
Copy link
Copy Markdown
Contributor

Respecto al carousel de categorías, necesito que crees una carpeta como alrededor de 15 imagenes relacionadas a medicamentos y las uses random en el listado. Esta solución es parcial porque lo deberiamos consumir desde el backend pero implica mucho tiempo hacer todos los cambios, entonces lo vamos a manejar así que es mas rapido @rikardodlpj88

Comment thread src/app/(shop)/page.tsx Outdated
Comment thread src/app/(shop)/search/page.tsx Outdated
Comment thread src/app/(shop)/search/page.tsx
@rikardodlpj88 rikardodlpj88 marked this pull request as draft May 4, 2025 00:54
@rikardodlpj88 rikardodlpj88 marked this pull request as ready for review May 5, 2025 05:37
@andres15alvarez andres15alvarez merged commit 8c142af into dev May 5, 2025
5 checks passed
@andres15alvarez andres15alvarez deleted the fix/ECOM/navigation-and-search-experience-improvements branch May 5, 2025 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants