Skip to content

Defer AutoFit execution to Save() for improved performance#1251

Merged
claudiamurialdo merged 2 commits intomasterfrom
fix/excel-auto-fit-performance
Mar 3, 2026
Merged

Defer AutoFit execution to Save() for improved performance#1251
claudiamurialdo merged 2 commits intomasterfrom
fix/excel-auto-fit-performance

Conversation

@claudiamurialdo
Copy link
Copy Markdown
Collaborator

Instead of calling Column.AutoFit() on every cell write, track dirty columns in a HashSet and apply AutoFit once per column at save time. Also fix fitColumnWidth() being called inside the loop in the Text setter.
Issue:207750

Defer AutoFit execution to Save() for improved performance
@genexusbot
Copy link
Copy Markdown
Collaborator

Cherry pick to beta success

@claudiamurialdo claudiamurialdo temporarily deployed to kafka-integration-tests February 13, 2026 16:28 — with GitHub Actions Inactive
@claudiamurialdo claudiamurialdo temporarily deployed to external-storage-tests February 13, 2026 16:28 — with GitHub Actions Inactive
@genexusbot
Copy link
Copy Markdown
Collaborator

Cherry pick to beta success

@claudiamurialdo claudiamurialdo merged commit bb4093b into master Mar 3, 2026
10 checks passed
@claudiamurialdo claudiamurialdo deleted the fix/excel-auto-fit-performance branch March 3, 2026 19:51
claudiamurialdo added a commit that referenced this pull request Mar 3, 2026
* Issue:207750
Defer AutoFit execution to Save() for improved performance

* Add missing using

---------

Co-authored-by: claudiamurialdo <c.murialdo@globant.com>
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