Skip to content

fix: preserve resized dimensions after updateOption - #5301

Merged
fangsmile merged 9 commits into
developfrom
fix/pr-5299-from-develop
Sep 2, 2026
Merged

fix: preserve resized dimensions after updateOption#5301
fangsmile merged 9 commits into
developfrom
fix/pr-5299-from-develop

Conversation

@fangsmile

Copy link
Copy Markdown
Contributor

Summary

  • Preserve manually resized pivot column widths after updateOption while still recalculating unresized columns.
  • Apply the same protection to row heights, including autoHeight visible-row expansion and fixed-height batch fill paths.
  • Rebuild the PR branch from latest develop and include the AIME-reviewed fixes from PR fix: preserve pivot column widths after updateOption #5299.

Test plan

  • npx jest __tests__/create-group-for-first-screen-issue-5298.test.ts --runInBand --silent=false
  • npm run compile
  • AIME review passed on equivalent final tree at commit 5c21a99bb

🤖 Generated with Claude Code

biubiukam and others added 6 commits August 31, 2026 11:12
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Keep row-height recomputation from touching manually resized rows and avoid using a manual row as the representative height for fixed body rows.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
fangsmile and others added 3 commits August 31, 2026 16:51
Keep defaultHeaderColWidth aligned with defaultColWidth when unset.
This prevents updateOption from resetting pivot row headers to 80.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Keep manually resized columns fixed during adaptive width redistribution.

Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com
Limit resized-column freezing to adaptive width mode.

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
@fangsmile
fangsmile merged commit 83017a7 into develop Sep 2, 2026
7 of 8 checks passed
@fangsmile
fangsmile deleted the fix/pr-5299-from-develop branch September 2, 2026 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants