Skip to content

refactor(aria/combobox): allow custom popup triggering logic (#32493)#33198

Open
tjshiu wants to merge 1 commit intoangular:mainfrom
tjshiu:customPopupTriggeringLogic
Open

refactor(aria/combobox): allow custom popup triggering logic (#32493)#33198
tjshiu wants to merge 1 commit intoangular:mainfrom
tjshiu:customPopupTriggeringLogic

Conversation

@tjshiu
Copy link
Copy Markdown
Contributor

@tjshiu tjshiu commented May 5, 2026

  • Adds trigger input to support custom trigger elements (like dropdown buttons).
  • Updates blur check logic to protect focus within custom popup triggering zones.
  • Introduces a new custom trigger button component example.

@pullapprove pullapprove Bot requested review from andrewseguin and ok7sai May 5, 2026 04:13
@tjshiu tjshiu added the dev-app preview When applied, previews of the dev-app are deployed to Firebase label May 5, 2026
@tjshiu tjshiu requested a review from adolgachev May 5, 2026 04:13
@tjshiu tjshiu added the target: major This PR is targeted for the next major release label May 5, 2026
…#32493)

- Adds `trigger` input to support custom trigger elements (like dropdown buttons).
- Updates blur check logic to protect focus within custom popup triggering zones.
- Introduces a new custom trigger button component example.
@tjshiu tjshiu force-pushed the customPopupTriggeringLogic branch from 726952e to 0a5903a Compare May 5, 2026 04:16
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 5, 2026

Deployed dev-app for 0a5903a to: https://ng-dev-previews-comp--pr-angular-components-33198-dev-t91qjucx.web.app

Note: As new commits are pushed to this pull request, this link is updated after the preview is rebuilt.

readonly openOnInput = input(true, {transform: booleanAttribute});

/** Optional trigger element associated with the combobox. */
readonly trigger = input<HTMLElement | undefined>(undefined);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I feel these two APIs are for niche use cases that need more discussion. Let's discuss in the sync.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah - I can see that: It's related to this: #32493

I added an example for dev-app preview, which adds a trigger button.

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

Labels

area: aria/combobox dev-app preview When applied, previews of the dev-app are deployed to Firebase target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants