Skip to content

CI v2: gridcraft PR 2495 @ c82a138 - #11

Open
FredApps wants to merge 9 commits into
ci/base-2495-67e39fafrom
ci/pr-2495-c82a138-v2-67e39fa-2f58c11ad934
Open

FredApps wants to merge 9 commits into
ci/base-2495-67e39fafrom
ci/pr-2495-c82a138-v2-67e39fa-2f58c11ad934

Conversation

@FredApps

Copy link
Copy Markdown
Owner

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

Original head: c82a138
Upstream target: 67e39fa
Test candidate: 0d1f3eb
Only workflow configuration differs from merged source tree 8d41f517bdb58746e4df0228d059772b33e5617b.
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