@web-padawan I can't figure out why the visual test on header row focus outline fails.

Updated grid screenshots. The outline seems to be rendered a bit differently - that could be the reason. This also affects the "row-dragover-on-top" screenshot. As for "dragstart" - the row height seems different (1px).

UPD: dragstart screenshot has 1px difference in row height, before it was 32px and now 33px which seems correct.

Had to do some more polishing to get all the fine details like I want them. But now there’s one more visual test that alludes me – can't figure out why the border between the last header row and first body row is missing when the header row is focused.

Something seems wrong regarding the empty state now:

 ❌ empty state > bounds > should not scroll horizontally with the columns
      AssertionError: expected -301 to be close to 0 +/- 1

Reverted changes to empty state that broke unit tests. There is still this problem remaining:

the border between the last header row and first body row is missing when the header row is focused.

UPD: the reason for it is this CSS:

  :host([navigating]) #header:focus-within,
  :host([navigating]) #footer:focus-within,
  [empty-state] #header {
    z-index: 2;
}

The fact that header is shown above overrides the border defined here:

```css
  table:has(#header > tr:not([hidden])) [part~='first-row-cell'] {
    border-top: var(--_row-border-width) solid var(--_border-color);
  }

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

The fact that header is shown above overrides the border defined here:

It really shouldn't, though. The header element shouldn't be above the first row when scroller to the top. I’m wondering if this is related to the transforms on the body rows, that there’s some sub-pixel alignment issue. I can't explain it in any other way.

Anyhoo, I fixed it by only changing the z-index of the header when the last row has focus within.

This ticket/PR has been released with Vaadin 25.0.0-alpha12.