Skip to content

fix: synchronize controlled layout before child render - #318

Merged
daniil-filippov-dev merged 1 commit into
mainfrom
CHARTS-13580_sync-controlled-layout
Sep 3, 2026
Merged

daniil-filippov-dev merged 1 commit into
mainfrom
CHARTS-13580_sync-controlled-layout

Conversation

@daniil-filippov-dev

Copy link
Copy Markdown
Contributor

What changed

  • synchronize externally controlled layout during render, before consumers receive new config
  • keep drag/resize baseline current across child layout effects
  • cover sequential controlled updates and layout-change callbacks with regression tests

Why

A controlled config update could reach children one render before its matching layout. Consumers then observed mismatched item IDs and missing layout entries.

Checks

  • focused Jest suite (13 tests)
  • TypeScript
  • ESLint
  • Prettier
  • build

@gravity-ui-bot

Copy link
Copy Markdown
Contributor

Preview is ready.

@daniil-filippov-dev
daniil-filippov-dev added this pull request to the merge queue Sep 3, 2026
Merged via the queue into main with commit ef23803 Sep 3, 2026
4 checks passed
@daniil-filippov-dev
daniil-filippov-dev deleted the CHARTS-13580_sync-controlled-layout branch September 3, 2026 09:04
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.

3 participants