Skip to content

DEV-26859 Replace exactly with exact for lineRule#73

Merged
ins426 merged 3 commits intodevelopfrom
DEV-26859
Apr 30, 2026
Merged

DEV-26859 Replace exactly with exact for lineRule#73
ins426 merged 3 commits intodevelopfrom
DEV-26859

Conversation

@ins426
Copy link
Copy Markdown

@ins426 ins426 commented Apr 29, 2026

According to the official OOXML specification, the valid value is exact (see https://c-rex.net/samples/ooxml/e1/Part4/OOXML_P4_DOCX_ST_LineSpacingRule_topic_ID0ERZ42.html ), but the OOXML documentation contains some ambiguity, as in certain sections the term exactly is mentioned (see https://c-rex.net/samples/ooxml/e1/Part4/OOXML_P4_DOCX_spacing_topic_ID0EA1OM.html). As a result, this led to set exactly as a possible value which it has no effect because it is not recognized as a valid value by the specification. That's why exactly is replaced with exact

@ins426 ins426 marked this pull request as ready for review April 29, 2026 14:23
Comment thread lib/properties/test/paragraph-properties.test.ts Outdated
Copy link
Copy Markdown

@harvestcore harvestcore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pre-approved. LGTM!

@ins426 ins426 merged commit 8957e62 into develop Apr 30, 2026
2 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.

4 participants