Skip to content

_PresentChoiceSetOperation doesn't remove keyboard listener in all cases #497

@crokita

Description

@crokita

Bug Report

The keyboard input listener has more cases where it gets attached to the rpc listeners than cases where it gets removed. This creates scenarios where listeners continue to be active after _PresentChoiceSetOperation (_PreloadPresentChoicesOperation in the next release) is finished. No request to activate the keyboard is needed. Multiple errors get logged on every keypress.

Reproduction Steps
  1. Send multiple valid choice sets
  2. Invoke screen manager's presentKeyboard
  3. Use the on-screen keyboard
Expected Behavior

Each button press should invoke a listener one time for every event

Observed Behavior

Each button press invokes multiple listeners for each event, depending on how many times choice sets were uploaded.

Browser & Version Information

1.4 on all environments.

Test Case, Sample Code, and / or Example App

Run the screen-choices test for the test suite in feature/release-tests

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions