Skip to content

[Bug]: ActionMenu's onChange handler invoked twice for larger touch devices #5556

Open
@croberts5

Description

@croberts5

Code of conduct

  • I agree to follow this project's code of conduct.

Impacted component(s)

ActionMenu

Expected behavior

The @change callback should be invoked once per action when using a larger touch screen device (ex: iPad).

Actual behavior

The @change callback is being invoked twice per action when using a larger touch screen device.

Might be related to:
#5511

Screenshots

No response

What browsers are you seeing the problem in?

No response

How can we reproduce this issue?

Here's a fork of the bug:
Components in this fork are on v1.5.0
https://stackblitz.com/edit/vitejs-vite-iysux6uy?file=src%2Fmy-element.ts,package.json,index.html

Open the console, and toggle the device toolbar. Pick iPad as the device.
Click the menu, select an item. Asset in the console that "changed!" is logged twice.
Components v1.4.0 or earlier do not show the double invocation of the @change callback.

Issue is reproducible using the device emulator in Chrome as well as using the iPad emulator on Mac.
I can be reached internally @connorr on slack.

Sample code or abstract reproduction which illustrates the problem

No response

Severity

SEV 3

Logs taken while reproducing problem

No response

Would you like to track this issue in Jira?

  • Yes, please tell me the ticket number!

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions