Skip to content

v5.5.12 - #644

Merged
kapantzak merged 2 commits into
masterfrom
fix/header-cell-filter-height
Sep 21, 2026
Merged

kapantzak merged 2 commits into
masterfrom
fix/header-cell-filter-height

Conversation

@kapantzak

Copy link
Copy Markdown
Contributor

Problem

Table columns that declare a header filter render the select outside the header cell, clipped from the bottom edge.

The compact 40px header row introduced in #641 was applied as a fixed height:

height="40px"

but the header cell stacks the label and the filter in a column flex, so a filtered column needs more than 40px and the select overflows.

Reproduced in @netdata/cloud-frontend on the space settings nodes table, where the Status and Commissioned columns are the two with meta.filter.component: "select". It affects every filtered column in every consuming table, including the notifications channels table.

Fix

- height="40px"
+ height={{ min: "40px" }}

A minimum rather than a fixed height. Label-only headers still render at exactly 40px, so the compact design from #641 is preserved; only cells that carry a filter grow.

Tests

Two regression tests in cell.test.js, both confirmed to fail with the fix reverted:

  • label-only headers stay at the compact 40px, so this cannot be "fixed" by returning to the old 45px
  • a header holding a filter is not constrained to 40px

Full suite: 99 suites, 487 tests, 9 snapshots, all passing.

The compact 40px header row was set as a fixed height, but the header
cell stacks the label and the column filter in a column flex, so any
column with a filter rendered its select outside the cell.

Setting it as a minimum keeps label-only headers at 40px and lets the
filtered ones grow to fit.
@kapantzak kapantzak changed the title v5.5.12 - let table header cells grow around a column filter v5.5.12 Sep 21, 2026
@kapantzak
kapantzak merged commit 1ee3f73 into master Sep 21, 2026
3 checks passed
@kapantzak
kapantzak deleted the fix/header-cell-filter-height branch September 21, 2026 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants