Uploaded image for project: 'OpenShift Virtualization'
  1. OpenShift Virtualization
  2. CNV-67857

PatternFly Chart hover broken due to delaunator package version conflict

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Undefined Undefined
    • CNV v4.21.0
    • CNV v4.19.0, CNV v4.20.0
    • CNV User Interface
    • None
    • Quality / Stability / Reliability
    • False
    • Hide

      None

      Show
      None
    • False
    • CNV v4.20.0.rhel9-135
    • None

      Description of problem:

      Hovering over charts does not display the expected label. Instead, an error "TypeError: _delaunator.default is not a constructor" is thrown. This is due to a version conflict between @patternfly/react-charts (which expects delaunator v4.0.1) and @patternfly/react-topology (which depends on delaunator v5.0.1).

      Version-Release number of selected component (if applicable):

       

      How reproducible:

      This issue is consistently reproducible.

      Steps to Reproduce:

      1. Navigate to a page containing a chart component that utilizes the @patternfly/react-charts library (For example - the overview section under Virtualization).  
      2. Hover your mouse over the chart. 
      

      Actual results:

      When hovering over a chart, a "TypeError: _delaunator.default is not a constructor" error is thrown, and no tooltip/label is displayed.
      In DEV - an error message is shown in the UI. In Prod - the error is suppressed and only shown in the dev console. In both cases the tooltip / label is not displayed. 

      Expected results:

      Expected results:
      The tooltip/label should appear on hover, displaying the data point details without any errors.

      Additional info:

       A temporary workaround is to restrict the delaunator dependency to version 4.0.1 in the project's package.json file.

              rh-ee-gkremer Gal Kremer
              rh-ee-gkremer Gal Kremer
              Guohua Ouyang Guohua Ouyang
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Created:
                Updated: