Esri / calcite-design-system

A monorepo containing the packages for Esri's Calcite Design System
https://developers.arcgis.com/calcite-design-system/
Other
269 stars 75 forks source link

[Input Date Picker] Multiple components with and without `range` can create buggy behavior #9167

Open geospatialem opened 2 months ago

geospatialem commented 2 months ago

Check existing issues

Actual Behavior

When multiple input-date-pickers with, or without range are active in an app, the component's can observe a buggy behavior when interacting with the date-picker.

For instance, with range active: picker-issue

Expected Behavior

Multiple input-date-pickers can be present in an app with no buggy behavior.

Reproduction Sample

Reproduction Steps

First sample:

  1. Open the sample
  2. Open the first input-date-picker
  3. Open the first range from the input-date-picker
  4. Open the second input-date-picker
  5. Observe the behavior
  6. The same process can be repeated with the second input-date-picker and attempting to open the first input-date-picker

Second sample:

  1. Open the sample
  2. Open the first input-date-picker
  3. Open the second input-date-0picker
  4. Observe the behavior

Reproduction Version

2.7.1

Relevant Info

Regression?

No response

Priority impact

p4 - not time sensitive

Impact

No response

Calcite package

Esri team

Calcite (dev)

geospatialem commented 2 months ago

The efforts of #3455 seem to resolve the behavior from above. Will confirm if the behavior is mitigated once the efforts of #3455 are merged - targeted for May 2024.

nwhittaker commented 2 months ago

@geospatialem, I'm seeing the same bug with two non-range pickers as well. I have a repro in case you want to roll that into the testing for this one, or I can create a separate issue.

geospatialem commented 2 months ago

@geospatialem, I'm seeing the same bug with two non-range pickers as well. I have a repro in case you want to roll that into the testing for this one, or I can create a separate issue.

@nwhittaker Good catch - updated the context of the issue above to include range and non-range and added your Codepen sample to the above for context and verification.