Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(ui5-avatar-group): click event is fired only once #3196

Merged
merged 7 commits into from
May 5, 2021

Conversation

dobrinyonkov
Copy link
Contributor

@dobrinyonkov dobrinyonkov commented Apr 29, 2021

Fixes: #3133

Summary:

  • keydown/keyup handlers are now used only when type is eqaul to Group and for Individual, the logic is handled in the _onClick and _onUI5Click methods
  • ui5-avatar fires ui5-click event on Space and Enter so no handling is requred in the keyup/keydown handlers of the group
  • ui5-button fires click event on Space and Enter - also, no handling requred from ui5-avatar-group

@CLAassistant
Copy link

CLAassistant commented Apr 29, 2021

CLA assistant check
All committers have signed the CLA.

@dobrinyonkov dobrinyonkov changed the title Avatar key down group event fix(ui5-avatar-group): click event is fired only once Apr 29, 2021
@fifoosid fifoosid merged commit 95986e2 into master May 5, 2021
@fifoosid fifoosid deleted the avatar-keyDown-group-event branch May 5, 2021 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG][UI5-AvatarGroup]: onClick event fired twice when use keyboard enter
5 participants