Skip to content

fix(ACI0104847,ACI0105646): preserve formatting in picture exports [ai] - #29

Open
e-marchand wants to merge 1 commit into
mainfrom
erm/aci0104847_aci0105646
Open

e-marchand wants to merge 1 commit into
mainfrom
erm/aci0104847_aci0105646

Conversation

@e-marchand

Copy link
Copy Markdown
Contributor

Summary

Extract the two related picture-export fixes from bf83478 in one commit because they modify the same export and rendering code. No bug-description Markdown or unrelated fixes are included.

  • ACI0104847: preserve number-format placeholder widths for alignment, overflow and shrink-to-fit rendering.
  • ACI0105646: include table and conditional-format visual styles, and extend the picture range to empty table cells.

Validation

JavaScript syntax passed. Executed Utils.addFormatedText with mocked SpreadJS inputs to check placeholder metadata, fill formats, actual-style differences, empty table rows, overlapping-range deduplication, bounded whole-column rules, conditional number formats and skipped data bars. All four extracted files match bf83478 exactly. JavaScript whitespace checks passed; 4D whitespace checks preserve the existing trailing-space convention.

Interactive 4D SVG rendering was not run. Please review; this PR is intentionally left unmerged.

@e-marchand e-marchand self-assigned this Oct 23, 2026
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