Skip to content

fix: preserve row thickTop - #359

Merged
MathNya merged 1 commit into
MathNya:masterfrom
developer0hye:fix/read-row-thick-top
Sep 3, 2026
Merged

fix: preserve row thickTop#359
MathNya merged 1 commit into
MathNya:masterfrom
developer0hye:fix/read-row-thick-top

Conversation

@developer0hye

Copy link
Copy Markdown
Contributor

Summary

  • store the OOXML row thickTop flag alongside thickBot
  • expose v3 thick_top / set_thick_top accessors and the deprecated compatibility getter
  • preserve the flag through XLSX read/write round trips

Verification

  • cargo test --test integration_test row_thick_top_roundtrips -- --exact
  • cargo test --lib (114 passed)
  • cargo +nightly fmt --check

Signed-off-by: Yonghye Kwon <developer.0hye@gmail.com>
@MathNya

MathNya commented Sep 3, 2026

Copy link
Copy Markdown
Owner

@developer0hye
Thank you for the PR.
Since there were no issues, I’ll go ahead and merge it.

@MathNya
MathNya merged commit 23eb791 into MathNya:master Sep 3, 2026
1 of 5 checks passed
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