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

Make Url-Mapping Case-Insensitive #1405

Closed
aedu-hostettler opened this issue May 12, 2023 · 0 comments · Fixed by #1435
Closed

Make Url-Mapping Case-Insensitive #1405

aedu-hostettler opened this issue May 12, 2023 · 0 comments · Fixed by #1435
Assignees
Labels
🐞 bug Something isn't working 📦 internet-header Related to the @swisspost/internet-header package
Milestone

Comments

@aedu-hostettler
Copy link

For a header-menu-item to be marked as active (bold-font, yellow underline), the current visited Url is analyzed and compared with the given header-config-url of a header-menu item.

It seems to do the comparison case-sensitive. Could you change it to case-insensitive?

Example:
the given Value of the configuration: some.com/Index
the entered url in browser: some.com/index

because the index is not capital case -> Index, the page is not recognised as the same, that is configured. and so the link in the menu does not get the ".active"-css class, thus the user feels, like the wrong page is shown, or the header is faulty.

@gfellerph gfellerph added 🐞 bug Something isn't working 📦 internet-header Related to the @swisspost/internet-header package labels May 30, 2023
@gfellerph gfellerph added this to the Maintenance milestone May 30, 2023
@gfellerph gfellerph self-assigned this May 31, 2023
@gfellerph gfellerph linked a pull request May 31, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug Something isn't working 📦 internet-header Related to the @swisspost/internet-header package
Projects
Development

Successfully merging a pull request may close this issue.

2 participants