Skip to content

Corrige problema al detectar la opción de navegación activa cuando se navega por hashes#155

Merged
francescarpi merged 1 commit intodevelopfrom
fix/138_menu_color
Mar 31, 2026
Merged

Corrige problema al detectar la opción de navegación activa cuando se navega por hashes#155
francescarpi merged 1 commit intodevelopfrom
fix/138_menu_color

Conversation

@francescarpi
Copy link
Copy Markdown
Collaborator

Antes:
image

Ahora:
image

Closes #138

- Compare full URLs (pathname + hash) instead of just pathname
- Add event listeners for hashchange, popstate, and menu clicks
- Fixes issue where multiple menu items with same path but different hashes were all marked active
- Ensures active state updates immediately when clicking hash-based navigation links
@francescarpi francescarpi added the bug Something isn't working label Mar 31, 2026
@francescarpi francescarpi self-assigned this Mar 31, 2026
Copy link
Copy Markdown
Collaborator

@ctrl-alt-d ctrl-alt-d left a comment

Choose a reason for hiding this comment

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

LGFM

@francescarpi francescarpi merged commit f193d22 into develop Mar 31, 2026
1 check passed
@francescarpi francescarpi deleted the fix/138_menu_color branch March 31, 2026 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants