orneryd / uiGrid / 26066082583

34%

push

github

fix(grid): persist framework slots across patch passes + UI polish

  - Patch path's per-row fingerprint short-circuit was skipping stageCell
    for unchanged rows, so frameworkSlots.flush() diffed an empty pending
    set against a full last set and emitted a spurious cellSlotsChanged
    removed for every cell. Angular embedded views and React portals were
    torn down on first paint and only reappeared after a scroll triggered
    a virtual-body rebuild that re-staged everything. Added
    FrameworkSlotBridge.carryRowCells(rowId) and called it from
    patchExistingRows when canSkip is true so the row's known slots carry
    forward and the diff is a no-op.

  - Move the row toggle (tree + expand) before .cell-content in the
    cell-shell so the disclosure chevron sits on the leading edge of the
    primary cell. Drop the `margin-inline-start: auto` override that
    previously pushed the expand toggle to the trailing edge. Updated the
    selection+expand correctness test to assert the new ordering.

  - Under-estimate the host's reported viewport width by 2px when feeding
    it into column-track resolution. The body viewport's overflow-x: auto
    plus sub-pixel rounding on the host box made tracks resolve ~1-2px
    wider than the actual renderable width, surfacing a spurious default
    horizontal scrollbar.

  - Add `isolation: isolate` and `contain: layout paint` on :host so
    sibling repaints (button hover transforms in the surrounding page)
    don't invalidate the grid's sticky filter strip. Fixes the filter-row
    flicker observed on the React and vanilla demo pages while Angular
    (whose host establishes its own stacking context via component view
    encapsulation) was already flicker-free.

  - Web-components demo: add a renewalDate column with an interactive
    date-picker slot template; wire a delegated change handler that walks
    back to the body cell, mutates the row in place, and re-publishes via
    setData() so the p... (continued)

1284 of 4462 branches covered (28.78%)

Branch coverage included in aggregate %.

2334 of 6178 relevant lines covered (37.78%)

17.57 hits per line