MenuOption's disabled Property Does Not Prevent Clicking When <router-link /> or <a /> is Used

[XiaoSong-CPE]

Describe the bug

When the disabled property is applied to a menu option rendered with RouterLink (as demonstrated in the documentation here), the appearance of the menu option changes correctly but it remains clickable.

This behaviour appears to be inconsistent. If RouterLink is not involved, both the icon and label require onClick to function, and the margin area cannot be clicked. It seems that n-menu is treating RouterLink differently, which we believe constitutes a bug and should be addressed.

Steps to reproduce

1. Use the code from the documentation (link) to render the menu.
2. Set the disabled attribute to true for the menu options.

(by the way, the sandbox link in that part of documentation doesn't work because the routes are unset)

Link to minimal reproduction

https://codesandbox.io/p/sandbox/focused-grothendieck-hynjrq

System Info

System:
    OS: Windows 11 10.0.26100
    CPU: (12) x64 12th Gen Intel(R) Core(TM) i5-12450H
    Memory: 3.23 GB / 15.73 GB
  Binaries:
    Node: 22.13.1 - C:\Program Files\nodejs\node.EXE
    npm: 10.9.2 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Chromium (131.0.2903.112)
    Internet Explorer: 11.0.26100.1882
  npmPackages:
    naive-ui: ^2.41.0 => 2.41.0
    vue: ^3.4.30 => 3.5.13

Used Package Manager

npm

Validations

• Read the Contributing Guidelines.
• Read the docs.
• Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
• Check that this is a concrete bug. For Q&A open a GitHub Discussion.
• The provided reproduction is a minimal reproducible example of the bug.

[XiaoSong-CPE]

The <a /> behaves the same as <router-link />.

[blackawn]

In my opinion, the effective a tag in the browser should be optimistic
So when you set this menu item to disabled, you can convert it to a basic tag.

menu.label = () =>
  disabled
    ? meta?.label
    : h(
        RouterLink,
        {
          to: { name },
        },
        { default: () => meta?.label },
      )