Skip to content

Conversation

akwasniewski
Copy link
Contributor

Description

On web, In EventManagers, the registerListeners was called twice for each handler of a Button

Test plan

  • Add a console.log to both registerListners() and unregisterListeners() in web/tools/KeyboardEventManager (or any other event manager)
  • Observe logs in the Web styles reset example in the example app

@akwasniewski akwasniewski requested a review from m-bert July 21, 2025 13:35
@akwasniewski akwasniewski changed the title fixed repeated listener registration [Web] fixed repeated listener registration Jul 21, 2025
@akwasniewski akwasniewski merged commit 2d19b81 into main Jul 21, 2025
1 check passed
@akwasniewski akwasniewski deleted the @akwasniewski/listeners-registered-repeatedly branch July 21, 2025 14:09
j-piasecki pushed a commit that referenced this pull request Jul 22, 2025
## Description

On web, In `EventManagers`, the `registerListeners` was called twice for
each handler of a `Button`

## Test plan

* Add a console.log to both `registerListners()` and
`unregisterListeners()` in `web/tools/KeyboardEventManager` (or any
other event manager)
* Observe logs in the `Web styles reset` example in the example app
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants