Block - manually setting to open still fires calciteBlockOpen event

[ian13262]

Check existing issues

• I have checked for existing issues to avoid duplicates

Actual Behavior

Setting a calcite-block to open manually (open = true) still fires the calciteBlockOpen event. Note that the deprecated event calciteBlockToggle is not fired in this case.

Expected Behavior

Setting open = true manually should not fire an open event. Specifically relevant when a block should be open on page load, but no event should occur as a result.

Reproduction Sample

https://codepen.io/iyouth66/pen/poYzyjp

Reproduction Steps

1.) Open DevTools
2.) Observe that the calciteBlockOpen event is called even when the property open is set to true manually (rather than actually clicking the calcite-block element).

Reproduction Version

2.0.0

Relevant Info

cc @AdelheidF

Regression?

No response

Priority impact

p3 - want for upcoming milestone

Impact

Specifically relevant when a block should be open on page load, but no event should occur. For example, opening the block by clicking might trigger a focus event, but this event should not be triggered if the block is just set to open on page load (maybe the focus should be elsewhere).

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 Online

[jcfranco]

This is behaving as expected, at least the way we've got open/close events set up. The reason we do this is because it is not possible to know exactly when the component is open when set programmatically, so the event is necessary. We've got a draft PR with an update to our event conventions on this.

We could look into providing info in the event to help distinguish open/close events between user and programmatically-triggered contexts.

[AdelheidF]

We could look into providing info in the event to help distinguish open/close events between user and programmatically-triggered contexts.

I think this would be useful.

[geospatialem]

We should work through some explorations and prototypes using a spike prior to team discussions that could be useful across the calcite components package and event types.