Skip to content

fix(aria/menu): unable to set softDisabled#33265

Merged
crisbeto merged 1 commit into
angular:mainfrom
crisbeto:33262/aria-menu-soft-disabled
May 16, 2026
Merged

fix(aria/menu): unable to set softDisabled#33265
crisbeto merged 1 commit into
angular:mainfrom
crisbeto:33262/aria-menu-soft-disabled

Conversation

@crisbeto
Copy link
Copy Markdown
Member

Fixes the following issues in the Aria menu:

  1. The menu didn't have a softDisabled input.
  2. The menu bar had a softDisabled input, but it was being ignored.

Fixes #33262.

@crisbeto crisbeto requested a review from ok7sai May 16, 2026 07:35
@crisbeto crisbeto added the target: patch This PR is targeted for the next patch release label May 16, 2026
@pullapprove pullapprove Bot requested a review from adolgachev May 16, 2026 07:35
Fixes the following issues in the Aria menu:
1. The menu didn't have a `softDisabled` input.
2. The menu bar had a `softDisabled` input, but it was being ignored.

Fixes angular#33262.
@crisbeto crisbeto force-pushed the 33262/aria-menu-soft-disabled branch from 3e92d66 to 3071ef6 Compare May 16, 2026 07:36
@crisbeto crisbeto added action: merge The PR is ready for merge by the caretaker target: rc This PR is targeted for the next release-candidate and removed target: patch This PR is targeted for the next patch release labels May 16, 2026
@crisbeto crisbeto merged commit 905aeb5 into angular:main May 16, 2026
28 of 30 checks passed
@crisbeto
Copy link
Copy Markdown
Member Author

This PR was merged into the repository. The changes were merged into the following branches:

crisbeto added a commit that referenced this pull request May 16, 2026
Fixes the following issues in the Aria menu:
1. The menu didn't have a `softDisabled` input.
2. The menu bar had a `softDisabled` input, but it was being ignored.

Fixes #33262.

(cherry picked from commit 905aeb5)
@Daherkarim
Copy link
Copy Markdown

Thank you for the swift fix, much appreciated.

I see some properties on the MenuPattern defined, hardcoded and not exposed.

I was wondering if this is intentional or also missed.

Thanks again :)

@crisbeto
Copy link
Copy Markdown
Member Author

crisbeto commented May 18, 2026

I think those were inherited from the list behavior and don't necessarily make sense to be configured in the menu.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: aria/menu target: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(aria/menu): Menu softDisabled input

3 participants