Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Datagrid column picker disappears when datagrid changes height #1475

Open
martinbrom opened this issue Jul 9, 2024 · 1 comment
Open

Datagrid column picker disappears when datagrid changes height #1475

martinbrom opened this issue Jul 9, 2024 · 1 comment

Comments

@martinbrom
Copy link
Contributor

This is a bug report for the @clr Angular or UI versions of the design system.
For the web-component implementation of Clarity (@cds), visit vmware-clarity/core.

If you are a VMware employee or a contractor in VMware, please use our support space in Google Chat to raise Clarity issues.

Describe the bug

When a column of a datagrid is hidden using the column picker, and the datagrid layout shifts as a result, the column picker will close itself.

Before hiding "tall column":
image

After hiding "tall column":
image

How to reproduce

https://stackblitz.com/edit/clarity-light-theme-clr17-7t9jcn

Steps to reproduce the behavior:

  1. Create a datagrid with a max-height set and hideable columns specified
  2. Define a column where elements have a bigger height than the other column cells
  3. Scroll to the bottom of the datagrid (the dg scrollbar has to be visible)
  4. Open column picker
  5. Hide the taller column
  6. Observe that the column picker disappears

Expected behavior

Column picker stays open

Versions

Clarity version:

"@cds/core": "^6.12.0",
"@clr/angular": "^17.2.1",
"@clr/ui": "^17.2.1",

Framework version:

"@angular/core": "~17.1.2",

Device:

  • Type: Lenovo ThinkPad P53
  • OS: Windows 10
  • Browser: Chrome
  • Version: 126.0.6478.127

Additional notes

none

@dtsanevmw dtsanevmw self-assigned this Oct 2, 2024
@dtsanevmw
Copy link
Contributor

Hello,

By design popovers close on scroll and there are these edge cases where scroll event is triggered from that change in height and it closes the popover. That's known issue and it's planned for future rework of popovers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants