[Rauneet-coder]

Description

Clicking <SideNavigationItem disabled onClick={handler}>Reports</SideNavigationItem> still invokes handler. The shared Item renders a disabled item as a div, but attaches onClick unconditionally.

Omit the callback when disabled. Add a small parameterized test in the existing navigation suite covering disabled and enabled clicks.

Validation

• Reproduced before the fix: disabled items invoke the callback with and without href.
• Verified after the fix using React 18/jsdom: disabled callbacks are suppressed and enabled callbacks still run. Six isolated checks also pass, including disabled-state changes and section toggling; the local harness substitutes the styling layer and UID hook.
• Prettier and git diff --check pass.
• The committed Testing Library test and full build/type checks could not be completed: npm registry DNS is unavailable and repository dependencies are missing. Direct TypeScript diagnostics are identical before and after the change.

Scope

Patch: Bug Fix