Navigation: Fix bug in calculating the active menu item

[aristath]

What?

Fixes #49153

Why?

As described in #49153, when a post and a term have the same ID and both exist in the menu, the code that determines the active menu item is wrong. As a result, both the term and the post get marked as active because we only check the ID and not the context.

How?

Tweaking the check to determine if a menu item is active.
$attributes['kind'] can be things like taxonomy, post-type etc. Using get_queried_object() in a term will return an object which will have the taxonomy defined. In a post, the post_type will be defined and so on.
This PR will check that the ID is the same, AND that the context (post/term etc) is right before marking a menu item as active.

Testing Instructions

1. On a fresh installation create some terms and some pages.
2. Find a page and a term that have the same ID, and add them both in a navigation block.

Before the PR, visiting the term will highlight both the post and the term as active, and the same will happen when visiting the page.
After the PR, the current-menu-item class will only be added to the right element.

Note: The same code was applied both to the navigation-link and the navigation-submenu blocks since they were using identical checks.

Testing Instructions for Keyboard

Not applicable.

[github-actions]

Flaky tests detected in aeb1335.
Some tests passed with failed attempts. The failures may not be related to this commit but are still reported for visibility. See the documentation for more information.

🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/4466778560
📝 Reported issues:

• [Flaky Test] displays the selected template name in the title for the header template #49019 in /test/e2e/specs/site-editor/title.spec.js

[Mamaduka]

The fix works as expected. Thank you, Ari!