Skip to content

calcite action with slot as header-menu-actions no longer calls onclick #8628

@previnWong

Description

@previnWong

Check existing issues

Actual Behavior

If you have a calcite-action with a slot as "header-menu-actions", the onclick event does not fire. If you change to "control" the onclick fires. But I lose the drop down menu that it creates.

click no longer works (header-menu-actions)
image

click works (control)
image

Expected Behavior

action with slot "header-menu-actions" use to fire the onclick event.

Reproduction Sample

https://codepen.io/pwongEsri/pen/MWxmbRr?editors=1000

Reproduction Steps

switch the slot in line 16 - from control to header-menu-actions to see the onclick event no longer fires

Reproduction Version

2.2.0

Relevant Info

Windows, Chrome

Regression?

worked on the version of calcite equal to JS Maps SDK 4.28

Priority impact

p2 - want for current milestone

Impact

Users cannot remove the item in the block, thus widget will be broken

Calcite package

  • @esri/calcite-components
  • @esri/calcite-components-angular
  • @esri/calcite-components-react
  • @esri/calcite-design-tokens
  • @esri/eslint-plugin-calcite-components

Esri team

ArcGIS Solutions

Metadata

Metadata

Labels

4 - verifiedIssues that have been tested, confirmed as mitigated, and are ready to close.ArcGIS Maps SDK for JavaScriptIssues logged by ArcGIS SDK for JavaScript team members.ArcGIS SolutionsIssues logged by ArcGIS Solutions team members.bugBug reports for broken functionality. Issues should include a reproduction of the bug.calcite-componentsIssues specific to the @esri/calcite-components package.estimate - 2Small fix or update, may require updates to tests.impact - p2 - want for an upcoming milestoneUser set priority impact status of p2 - want for an upcoming milestonep - highIssue should be addressed in the current milestone, impacts component or core functionality

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions