Skip to content

Fix RTF antiqueWhite preset color#81

Merged
DmySyz merged 1 commit into
Euro-Office:mainfrom
vixco:fix/rtf-antique-white-preset
Jun 2, 2026
Merged

Fix RTF antiqueWhite preset color#81
DmySyz merged 1 commit into
Euro-Office:mainfrom
vixco:fix/rtf-antique-white-preset

Conversation

@vixco
Copy link
Copy Markdown
Contributor

@vixco vixco commented May 29, 2026

Summary

  • Correct the RTF preset color name from aniqueWhite to antiqueWhite.
  • Keep RTF preset parsing/writing consistent with the OOXML preset color mappings already present in core.

Why

The RTF color helper used a misspelled preset name for RGB 250,235,215. As a result, valid antiqueWhite preset input was not recognized by this mapping, and serializing that color emitted the non-standard aniqueWhite name.

Validation

  • Ran git diff --check.
  • Confirmed aniqueWhite no longer appears in the repository.
  • Confirmed the RTF mapping now matches the existing OOXML antiqueWhite mappings.

Signed-off-by: vixco <93612321+vixco@users.noreply.github.com>
@vixco vixco marked this pull request as ready for review May 29, 2026 22:53
@vixco vixco requested a review from a team as a code owner May 29, 2026 22:53
@vixco vixco requested review from Aiiaiiio and removed request for a team May 29, 2026 22:53
@DmySyz DmySyz merged commit 1486595 into Euro-Office:main Jun 2, 2026
1 check passed
@DmySyz
Copy link
Copy Markdown
Contributor

DmySyz commented Jun 2, 2026

Thank you for the contribution!

@vixco
Copy link
Copy Markdown
Contributor Author

vixco commented Jun 2, 2026

Thank you for the contribution!

No problem at all. Great product

@juliusknorr juliusknorr added the bug Something isn't working label Jun 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants