Skip to content

CI v2: gridcraft PR 2495 @ c82a138 - #13

Open
FredApps wants to merge 9 commits into
ci/base-2495-a85b1aefrom
ci/pr-2495-c82a138-v2-a85b1ae-ac9b548f3584
Open

FredApps wants to merge 9 commits into
ci/base-2495-a85b1aefrom
ci/pr-2495-c82a138-v2-a85b1ae-ac9b548f3584

Conversation

@FredApps

Copy link
Copy Markdown
Owner

Fork validation for upstream gridcraft pull request number 2495. Never merge.

Original head: c82a138
Upstream target: a85b1ae
Test candidate: c1f92e4
Only workflow configuration differs from merged source tree baab1c09694e50f765a3f20b9cb06f4c67d860f5.
Policy: review-ci-v2.2-no-sonar. SonarCloud omitted by user instruction.

Harrison-Booth and others added 9 commits October 23, 2026 16:32
.xls (Excel 97-2003) import used only each cell's number format. It now
also reads the BIFF8 formatting records (Font, XF, Palette and Style),
so opened sheets keep their fonts, solid and pattern fills, borders
(including diagonals), alignment (rotation, indent, wrap, shrink),
locked and hidden protection, the workbook's custom colour palette, its
Normal font as the default, and its user-defined cell styles.

The decoding lives in a new xls/styles.rs; xls/mod.rs gains the record
arms and the build call. Short or malformed formatting records keep the
fields present and add one warning instead of refusing the workbook.
Tests build the records in code, including truncation and byte mutation.

Also merges origin/main, keeping its dialog-sheet, WINDOW2 and XF
number-format handling; the XF arm now goes through the new styles
module, which covers the number format too.
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.

2 participants