Skip to content

Fixes trigger errors with the passkeys modal#18612

Draft
gcamacho079 wants to merge 4 commits into5.xfrom
fix/passkey-modal-trigger
Draft

Fixes trigger errors with the passkeys modal#18612
gcamacho079 wants to merge 4 commits into5.xfrom
fix/passkey-modal-trigger

Conversation

@gcamacho079
Copy link
Contributor

Description

  • Moved setting the trigger element above the if (container) conditional, since the show() might set focus before we can grab the trigger element
  • Refactors some code in the Garnish activate setup that prevented focus being set on the button element specifically when clicked

Related issues

#18603

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant