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

Navigation doesn't always highlight selection #3509

Closed
deboer-tim opened this issue Aug 11, 2023 · 0 comments · Fixed by #3510
Closed

Navigation doesn't always highlight selection #3509

deboer-tim opened this issue Aug 11, 2023 · 0 comments · Fixed by #3510
Assignees
Labels

Comments

@deboer-tim
Copy link
Collaborator

Bug description

I noticed that the main navigation bar isn't always highlighting the selected item:

  • Docker extensions that have a space in the name (encoded URL) are never selected.
  • Settings isn't showing selection (was worked via a special case and regressed when making a component via chore: navitem component #3404).

Operating system

macOS 13.5

Installation Method

None

Version

next (development version)

Steps to reproduce

Install a Docker extension with a space like 'Disk Usage'. Selecting it or Settings in the main nav won't show the normal purple selection edge.

Relevant log output

No response

Additional context

No response

@deboer-tim deboer-tim added the kind/bug 🐞 Something isn't working label Aug 11, 2023
@deboer-tim deboer-tim self-assigned this Aug 11, 2023
deboer-tim added a commit that referenced this issue Aug 11, 2023
As per issue #3509, the main nav wouldn't show items as selected if they had a
space in the URL, or were child pages like settings/resources. This fixes the
first issue through URL encoding the href, and the latter by adding back the
startsWith() that was special-cased for settings before. New tests added for
both cases.

Signed-off-by: Tim deBoer <git@tdeboer.ca>
deboer-tim added a commit that referenced this issue Aug 15, 2023
As per issue #3509, the main nav wouldn't show items as selected if they had a
space in the URL, or were child pages like settings/resources. This fixes the
first issue through URL encoding the href, and the latter by adding back the
startsWith() that was special-cased for settings before. New tests added for
both cases.

Signed-off-by: Tim deBoer <git@tdeboer.ca>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants