Skip to content

refactor: grid internals and controllers - #80

Merged
rkaraivanov merged 1 commit into
masterfrom
rkaraivanov/grid-refactor
Aug 25, 2026
Merged

refactor: grid internals and controllers#80
rkaraivanov merged 1 commit into
masterfrom
rkaraivanov/grid-refactor

Conversation

@rkaraivanov

Copy link
Copy Markdown
Member

Reduce duplication and indirection across /src with no behavior or public API changes:

  • Factor the shared emit -> commit -> emit flow in FilterController into a single helper.
  • Collapse the six navigation handlers into two clamped movement primitives.
  • Extract the slotted-column scan in the grid, shared by slot change and column detection.
  • Add resolveCondition() to replace the repeated operand lookup casts in grid, filter controller and filter row.
  • Unify the active/inactive chip rendering in the filter row.
  • Drop needless indirection: pipeline getters, tree iterator generator, sort direction Map, first-render branch in partMap.
  • Flatten nested ternaries in header sort rendering and the filter row state preview; extract a shared adopt-styles predicate.

Reduce duplication and indirection across /src with no behavior
or public API changes:

- Factor the shared emit -> commit -> emit flow in FilterController
  into a single helper.
- Collapse the six navigation handlers into two clamped movement
  primitives.
- Extract the slotted-column scan in the grid, shared by slot change
  and column detection.
- Add resolveCondition() to replace the repeated operand lookup casts
  in grid, filter controller and filter row.
- Unify the active/inactive chip rendering in the filter row.
- Drop needless indirection: pipeline getters, tree iterator
  generator, sort direction Map, first-render branch in partMap.
- Flatten nested ternaries in header sort rendering and the filter
  row state preview; extract a shared adopt-styles predicate.
@rkaraivanov
rkaraivanov merged commit 571e29e into master Aug 25, 2026
4 checks passed
@rkaraivanov
rkaraivanov deleted the rkaraivanov/grid-refactor branch August 25, 2026 15:06
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.

1 participant