vmware-clarity / core

Clarity is a scalable, accessible, customizable, open-source design system built with web components. Works with any JavaScript framework, created for enterprises, and designed to be inclusive.
https://clarity.design
MIT License
164 stars 42 forks source link

Resizing column in grid doesn't work, shows error #216

Closed squidjam closed 1 year ago

squidjam commented 1 year ago

Describe the bug

When one clicks on the resizing control (the separator between the data grid columns' headers) an error is shown in the console and nothing else happens:

    Uncaught TypeError: Cannot read properties of undefined (reading 'find')
    at s$e.start (4c242522.js:44:31253)
    at host.addEventListener.passive (4c242522.js:44:31213)

How to reproduce

Follow this link to your own documentation and test it out: https://storybook.core.clarity.design/?path=/docs/internal-stories-grid--column-resize

Steps to reproduce the behavior:

  1. Go to your documentation
  2. Click on the resizing handle
  3. See error

Expected behavior

The column should resize and no error should be displayed in the console

Versions

Clarity project:

Clarity version:

Framework:

Framework version: React 18 and core's storybook

Device:

Additional notes

Why is there so little documentation around the data grid? (and for React there's none at all)

kevinbuhmann commented 1 year ago

Thanks for this bug report. I have opened a PR to fix this error.

Why is there so little documentation around the data grid?

The Clarity Core datagrid is incomplete. It's currently released as a preview.

github-actions[bot] commented 1 year ago

:tada: This issue has been resolved in version 6.3.2 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

github-actions[bot] commented 1 year ago

Hi there 👋, this is an automated message. To help Clarity keep track of discussions, we automatically lock closed issues after 14 days. Please look for another open issue or open a new issue with updated details and reference this one as necessary.