feat(NcAppNavigationItem): use exact route matching

[wofferl]

Resolves: #7965

updated

This PR changes the state to use from isActive to isExactActive and made active an optional prop to overwrite the default behaviour.

old

exact - Pass in true if you want the matching behaviour to be non-inclusive: https://router.vuejs.org/api/#exact

The exact prop for NcAppNavigationItem was removed in v9 without replacement.

The news app could now use this for a new feature (nextcloud/news#3148), to use /starred for all items and /starred/:feedId for sub items.

This PR adds the exact prop to control which active state - isExactActive or isActive - to use.

isExactActive: true if the [exact active class](https://v3.router.vuejs.org/api/#exact-active-class) should be applied. Allows to apply an arbitrary class

🏁 Checklist

• ⛑️ Tests are included or are not applicable
• 📘 Component documentation has been extended, updated or is not applicable

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 52.77%. Comparing base (dbbdf69) to head (f731b21).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #7939   +/-   ##
=======================================
  Coverage   52.77%   52.77%           
=======================================
  Files         103      103           
  Lines        3348     3348           
  Branches      976      977    +1     
=======================================
  Hits         1767     1767           
  Misses       1333     1333           
  Partials      248      248           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[ShGKme]

In v8 there was exact prop because NcAppNavigationItem is a wrapper over RouterLink which had the exact prop in the past in Vue 2. In other words, removed NcAppNavigationItem's exact prop followed removed <RouterLink>'s exact prop.

This prop was removed in Vue Router v4 (Vue 3) ~5 years ago: https://github.com/vuejs/rfcs/blob/master/active-rfcs/0028-router-active-link.md

In my opinion, we should not bring back a prop removed in the base component.

Also, isExactActive is different from old exact. It acts as exact-path, ignoring query and hash.

Adding a prop with the same name but different behaviour seems even more confusing to me and harder to catch. This means that a prop is not removed but changed between v8 and v9, which is still a breaking change from v8, but different from mentioned in the breaking change notes.

In the current case, I'd use isExactActive instead of isExact without additional configuration to always the the full path.

In addition, I'd make active optional but allow override router's active by active prop. So a developer can manually specify when a link is active in a more complex case (for example, with query/hash).

[wofferl]

@ShGKme can you review the requested changes
Since the News App has the feature that requires this fix in its beta version, it would be great if this fix could be included in 9.4.0