Skip to content

[WC-3280] DG: Fix unwanted scrollbar for cleared grids#2113

Open
yordan-st wants to merge 3 commits intomainfrom
fix/WC-3280_DG-custom-content-bug
Open

[WC-3280] DG: Fix unwanted scrollbar for cleared grids#2113
yordan-st wants to merge 3 commits intomainfrom
fix/WC-3280_DG-custom-content-bug

Conversation

@yordan-st
Copy link
Contributor

@yordan-st yordan-st commented Feb 25, 2026

Pull request type

Bug fix (non-breaking change which fixes an issue)


Description

  • Clearing a grid to 0 rows, would cause a unwanted recalculation of column widths, causing columns to overflow and in turn add scrollbars. This fix is aimed at making sure that: for empty grids the grid columns use the natural css grid layout instead of recalculating the column widths.
  • Setting the datagrid design property to striped would follow an incorrect order of 'striping' each odd row, starting with the second instead of the first row. This was caused by DOM changes introduced in previous DW versions. This is addressed with improved scss selectors.

What should be covered while testing?

  • Add a tooltip with a small image in the header filter
  • Set the width of that column to auto-fit content and for min-width property try both auto and set by content
  • Observe if the column width change weirdly when adding / deleting a row.
  • Observe the same if you add / delete and then leave and return to the page.

@yordan-st yordan-st changed the title fix: keep custom content column headers visible when grid becomes empty [WC-3280] DG: Fix unwanted scrollbar for cleared grids Feb 25, 2026
@yordan-st yordan-st force-pushed the fix/WC-3280_DG-custom-content-bug branch from 7daf508 to c7544b9 Compare February 26, 2026 14:56
@yordan-st yordan-st marked this pull request as ready for review February 26, 2026 14:56
@yordan-st yordan-st requested a review from a team as a code owner February 26, 2026 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant