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 :)

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