feat(numpages): apply field-switch formatting to total page count fields (SD-3327)#3637
Merged
luccas-harbour merged 16 commits intoJun 5, 2026
Conversation
superdoc-bot
Bot
added
review: thorough
review: careful
and removed
review: thorough
labels
Contributor
|
I attempted to verify against the spec via the ecma-spec MCP tools, but the calls were blocked (permission not granted in this session), so I've fallen back to ECMA-376 Part 1 §17.16.4 (field switches) and §17.16.5 (field types) directly. Here's the review. Status: PASS The changed handler (
No invented OOXML attributes/elements. The new No missing-required-attribute, wrong-default, or non-existent-element issues found on the OOXML boundary. One caveat worth flagging for the record: I couldn't get the ecma-spec tools to run (each call returned "permission not granted"), so the above rests on my own reading of §17.16.4. If you re-run with the spec tools authorized, the one thing worth a direct confirmation is the exact |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: efb1240066
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
luccas-harbour
force-pushed
the
luccas/sd-3327-feature-total-page-count-fields
branch
from
d102150 to
33fab23
Compare
Extend the shared page-number field format contract with ordinal and numericPicture metadata for later NUMPAGES parser/formatter steps. Add the hidden total-page-number PM attr, expose it on the typed node attrs, and thread it through the layout adapter format mapper with focused coverage. Widen the layout engine page-number state annotation to the shared PageNumberFormat type so the expanded contract remains compile-compatible.
Extend parsePageNumberFieldSwitches so Ordinal general switches map to the new ordinal page-number format and non-empty non-zero numeric pictures are retained in pageNumberNumericPicture instead of being discarded. Keep all-zero numeric pictures on the existing zero-padding path for backwards compatibility, and parse switch arguments from the uncollapsed instruction text so quoted picture literals keep their internal whitespace.
Extend the shared page-number formatter with English ordinal output and numeric-picture dispatch for preserved NUMPAGES switches. Numeric pictures now take precedence over enum formats and zero padding, while ordinal and existing page-number formats continue through the shared enum formatter. Focused contracts tests cover ordinal suffix rules, numeric-picture dispatch, and page-number normalization.
Route shape/textbox NUMPAGES through the existing simple pageNumberFormat enum path, matching the SECTIONPAGES branch without adding shape numeric-picture or zero-padding support. Format fields.rebuild total-page-number output with formatPageNumberFieldValue using preserved pageNumberFormat, pageNumberZeroPadding, and pageNumberNumericPicture attrs, and cover rebuilt resolvedText/text content for enum and numeric-picture cases.
Parse raw NUMPAGES instructions during fields.insert and pass the resulting page-number attrs into total-page-number creation. This preserves numeric picture and general format switches for header/footer NUMPAGES insertion without changing the existing body-insertion restriction.
Add focused coverage for complex and fldSimple NUMPAGES imports, numeric-picture PM/export round-trip, live total-page-count formatting, and shape NUMPAGES formatting. Also thread numeric-picture attrs through the existing export formatter and align the single-block totalPageCount resolver with the formatted runtime path so the new tests exercise the intended behavior.
Format total-page-number refreshes in FieldUpdate with the same preserved page-number metadata used by rebuild/export paths. Adds regression coverage for numeric-picture NUMPAGES updates.
Prefer stored total-page-number instructions during field discovery so fields.list/get report switched NUMPAGES instructions preserved by insert/import.
Apply simple pageNumberFormat handling to NUMPAGES in the legacy SVG text helper, matching the DomPainter shape path and existing PAGE/SECTIONPAGES behavior.
Parse NUMPAGES insert instructions from the original raw instruction so quoted numeric-picture literals keep significant internal whitespace.
Apply preserved NUMPAGES formatting in header/footer edit-mode NodeViews and cached DOM refreshes so switched totals match layout/export behavior.
Join header/footer instrText fragments verbatim so split quoted NUMPAGES numeric pictures do not gain synthetic spaces before parsing.
Keep significant whitespace inside quoted NUMPAGES numeric-picture instructions while still normalizing field-code whitespace outside quotes.
Route NUMPAGES formatting call sites through the existing getPageNumberFieldFormat helper and make zeroPadding handling consistent.
Add regression coverage for no-space split instrText switches, ordinal shape NUMPAGES rendering, and three-digit teen ordinal suffixes.
luccas-harbour
force-pushed
the
luccas/sd-3018-feature-sequence-fields
branch
from
759f6f7 to
28ee0e4
Compare
luccas-harbour
force-pushed
the
luccas/sd-3327-feature-total-page-count-fields
branch
from
33fab23 to
be44072
Compare
luccas-harbour
merged commit 6dd2c87
into
luccas/sd-3018-feature-sequence-fields
39 checks passed
luccas-harbour
added a commit
that referenced
this pull request
Jun 5, 2026
…-3018) (#3636) * feat(seq-fields): add shared SEQ instruction parser Add a shared SEQ instruction parser for later import, layout, API, rebuild, and F9 paths. The parser preserves raw instruction text, normalizes only field dispatch, handles quoted and unquoted arguments, parses SEQ-specific switches, records unknown switches, and reuses the existing page-number general-format mapping instead of copying format tables. Parser tests cover the Phase 1 required cases plus pair-reviewer fixes for raw instruction preservation, shared keyword dispatch, and quoted-only unescaping. * feat(seq-fields): import SEQ fields case-insensitively Route SEQ field preprocessing case-insensitively and add a v2 sequenceField importer before passthrough so synthetic sd:sequenceField nodes become PM sequenceField nodes instead of being dropped. Reuse the shared SEQ instruction parser in the sequence field translator, carry parsed import attrs onto the PM node, mark imported cached results stale, and keep raw instruction export behavior unchanged. Add focused preprocessing/import/translator tests covering uppercase and lowercase complex and fldSimple SEQ fields plus cached result preservation. * feat(seq-fields): add shared SEQ evaluator Add a pure SequenceFieldEvaluator next to the shared SEQ parser so layout, API, and update paths can share counter semantics without importing editor or rendering internals. The evaluator owns per-identifier counters, heading serial tracking for \s resets, explicit \r reset precedence, conservative field-argument fallback behavior, hidden-result handling, and shared contract-based number formatting. Add focused coverage for the required Phase 3 numbering, reset, repeat-current, hidden, formatting, empty-identifier, field-argument, and initial-counter cases. * feat(seq-fields): resolve SEQ display during layout Add SEQ token metadata to TextRun and emit sequence-field layout tokens without relying on a fake zero placeholder. Resolve SEQ display in a post-assembly toFlowBlocks pass using the shared SequenceFieldEvaluator so paragraph, table, list-compatible, and cache-hit blocks are renumbered in document order before layout measurement. Cover numbering modes, formatting, heading-level restarts, story isolation, table traversal, and FlowBlockCache renumbering with focused layout-adapter tests. * feat(seq-fields): recompute SEQ fields for API updates Add a shared transaction helper that recomputes body SEQ fields in ProseMirror state using the shared parser/evaluator and style-aware heading resolution from the layout adapter when converter style data is available. Wire raw field insertion, field rebuild, caption insertion/configuration, and F9 updates through the helper while preserving existing TOC and document-stat dispatch behavior. Field discovery now reads sequenceField.resolvedNumber as the resolved display text. Cover updater scopes, caption/field wrapper recomputation, field resolver readback, and F9 SEQ refresh behavior with focused tests. * fix(seq-fields): export current SEQ results Update sequenceField export so current evaluated resolvedNumber values are authoritative, including empty current results for hidden fields. Preserve imported cached child content when results are not current, with a fallback to non-current resolvedNumber only when no cached child content exists. Add export-routing tests for current result runs, hidden empty output, cached-content preservation, resolvedNumber fallback, and raw instructionTokens preservation. Add PM-updater coverage documenting Phase 7 conservative field-argument behavior: cached text wins, no-cache references repeat the previous counter, and counters do not advance. * fix(seq-fields): parse attached restart switches Handle Word-emitted SEQ switches such as \r0 and \s1 by normalizing attached numeric values through the existing restart parser path. This fixes the NDA browser-test case where hidden level reset fields imported with restartNumber null, causing the first visible level2 SEQ field to render as 2 instead of 1. Adds parser coverage for attached restart switches and a layout regression for hidden restart-zero fields seeding the next visible value. * fix(seq-fields): recompute F9 after TOC state changes Build the stat/SEQ update transaction from the current editor state after TOC updates dispatch their own transactions. This preserves TOC edits and shifted SEQ positions when F9 updates both TOC and SEQ fields. * fix(seq-fields): detect lowercase caption SEQ fallback Use the shared case-insensitive SEQ instruction check when caption discovery falls back to sequenceField nodes. This keeps imported lowercase seq captions discoverable by the API. * fix(seq-fields): restore legacy format default Keep the sequenceField schema default at ARABIC for backward compatibility while the shared parser continues to default parsed instructions to Arabic. * fix(seq-fields): preserve F9 SEQ selection across TOC edits Remember whether the original F9 selection contained a SEQ field before TOC updates dispatch their own transactions. This lets the fresh post-TOC transaction recompute SEQ fields even when TOC edits shift the selected field position. * refactor(seq-fields): share parsed attr projection Add sequenceFieldAttrsFromParsed beside the SEQ parser and use it from import, raw field insertion, caption insertion/configuration, and PM recompute. This keeps parsed instruction attrs and null/default normalization in one place. * fix(converter): parse attached SEQ format switches * fix(measuring): skip empty text runs during paragraph measurement Empty text runs were flowing into the segment/space accounting, where a trailing run boundary could be counted as a justification space and an oversized empty run's font size could inflate the visible line height. Short-circuit empty runs after recording font state so they no longer produce phantom spaces, segments, or height. * feat(numpages): apply field-switch formatting to total page count fields (SD-3327) (#3637) * feat(numpages): add numeric picture field metadata Extend the shared page-number field format contract with ordinal and numericPicture metadata for later NUMPAGES parser/formatter steps. Add the hidden total-page-number PM attr, expose it on the typed node attrs, and thread it through the layout adapter format mapper with focused coverage. Widen the layout engine page-number state annotation to the shared PageNumberFormat type so the expanded contract remains compile-compatible. * feat(numpages): preserve numeric picture switches Extend parsePageNumberFieldSwitches so Ordinal general switches map to the new ordinal page-number format and non-empty non-zero numeric pictures are retained in pageNumberNumericPicture instead of being discarded. Keep all-zero numeric pictures on the existing zero-padding path for backwards compatibility, and parse switch arguments from the uncollapsed instruction text so quoted picture literals keep their internal whitespace. * feat(numpages): format ordinal and picture counts Extend the shared page-number formatter with English ordinal output and numeric-picture dispatch for preserved NUMPAGES switches. Numeric pictures now take precedence over enum formats and zero padding, while ordinal and existing page-number formats continue through the shared enum formatter. Focused contracts tests cover ordinal suffix rules, numeric-picture dispatch, and page-number normalization. * fix(numpages): format rebuild and shape results Route shape/textbox NUMPAGES through the existing simple pageNumberFormat enum path, matching the SECTIONPAGES branch without adding shape numeric-picture or zero-padding support. Format fields.rebuild total-page-number output with formatPageNumberFieldValue using preserved pageNumberFormat, pageNumberZeroPadding, and pageNumberNumericPicture attrs, and cover rebuilt resolvedText/text content for enum and numeric-picture cases. * fix(numpages): preserve inserted field switches Parse raw NUMPAGES instructions during fields.insert and pass the resulting page-number attrs into total-page-number creation. This preserves numeric picture and general format switches for header/footer NUMPAGES insertion without changing the existing body-insertion restriction. * test(numpages): cover switched field evaluation Add focused coverage for complex and fldSimple NUMPAGES imports, numeric-picture PM/export round-trip, live total-page-count formatting, and shape NUMPAGES formatting. Also thread numeric-picture attrs through the existing export formatter and align the single-block totalPageCount resolver with the formatted runtime path so the new tests exercise the intended behavior. * fix(numpages): preserve F9 field formatting Format total-page-number refreshes in FieldUpdate with the same preserved page-number metadata used by rebuild/export paths. Adds regression coverage for numeric-picture NUMPAGES updates. * fix(numpages): expose preserved field instructions Prefer stored total-page-number instructions during field discovery so fields.list/get report switched NUMPAGES instructions preserved by insert/import. * fix(numpages): format legacy SVG shape totals Apply simple pageNumberFormat handling to NUMPAGES in the legacy SVG text helper, matching the DomPainter shape path and existing PAGE/SECTIONPAGES behavior. * fix(numpages): preserve inserted picture whitespace Parse NUMPAGES insert instructions from the original raw instruction so quoted numeric-picture literals keep significant internal whitespace. * fix(numpages): format edit-mode total pages Apply preserved NUMPAGES formatting in header/footer edit-mode NodeViews and cached DOM refreshes so switched totals match layout/export behavior. * fix(numpages): preserve split header field pictures Join header/footer instrText fragments verbatim so split quoted NUMPAGES numeric pictures do not gain synthetic spaces before parsing. * fix(numpages): preserve quoted picture instructions Keep significant whitespace inside quoted NUMPAGES numeric-picture instructions while still normalizing field-code whitespace outside quotes. * refactor(numpages): share field format attrs mapper Route NUMPAGES formatting call sites through the existing getPageNumberFieldFormat helper and make zeroPadding handling consistent. * test(numpages): cover ordinal and split switch edges Add regression coverage for no-space split instrText switches, ordinal shape NUMPAGES rendering, and three-digit teen ordinal suffixes. * fix(converter): preserve split page field switch args
harbournick
pushed a commit
that referenced
this pull request
Jun 5, 2026
…ers (SD-3007) (#3632) * feat(page-reference): resolve PAGEREF fields to live target page numbers Wire PAGEREF field references through to dynamic resolution against the paginated layout, so cross-references display the target's actual page number, format, and relative position. - Parse PAGEREF instructions into typed switches (\h, \p, \* general format, \# numeric picture, CHARFORMAT/MERGEFORMAT) in a shared pageref-instruction parser; store results as typed node attrs. - Add buildPageRefAnchorMap to locate bookmark targets within the resolved layout (fragment PM ranges, with paragraph/table run-range fallbacks) and expose target display page metadata. - Resolve PAGEREF run text in resolveLayout: format the target display number via numeric picture / general format, emit relative-position text ("on page N", "above"/"below"), and bump the paint cache version when the resolved page changes. - Add formatIntegerWithNumericPicture for the Word \# picture subset (placeholders, grouping, sign slots, fractional, section selection). - Preserve instruction tokens for lossless export and capture the first instrText run rPr for CHARFORMAT field run properties. - Gate body page-token resolution behind SD_BODY_PAGE_TOKENS. * fix(page-reference): locate list item anchors * fix(page-reference): resolve table cell refs * fix(page-reference): resolve list item refs * fix(page-reference): preserve refs on repaint * fix(page-reference): apply charformat props * fix(page-reference): export charformat props * fix(page-reference): scope table anchors to fragments * fix(page-reference): scope split row anchors * fix(page-reference): update nested line slices * fix(page-reference): ignore anchor gaps * fix(page-reference): avoid split-line duplication * fix(page-reference): allow nearby bookmark markers * test(page-reference): cover planned pageref gaps * docs(page-reference): document pageref constraints * fix(page-reference): decouple bookmarks from page token flag * feat(seq-fields): import, resolve, and export SEQ sequence fields (SD-3018) (#3636) * feat(seq-fields): add shared SEQ instruction parser Add a shared SEQ instruction parser for later import, layout, API, rebuild, and F9 paths. The parser preserves raw instruction text, normalizes only field dispatch, handles quoted and unquoted arguments, parses SEQ-specific switches, records unknown switches, and reuses the existing page-number general-format mapping instead of copying format tables. Parser tests cover the Phase 1 required cases plus pair-reviewer fixes for raw instruction preservation, shared keyword dispatch, and quoted-only unescaping. * feat(seq-fields): import SEQ fields case-insensitively Route SEQ field preprocessing case-insensitively and add a v2 sequenceField importer before passthrough so synthetic sd:sequenceField nodes become PM sequenceField nodes instead of being dropped. Reuse the shared SEQ instruction parser in the sequence field translator, carry parsed import attrs onto the PM node, mark imported cached results stale, and keep raw instruction export behavior unchanged. Add focused preprocessing/import/translator tests covering uppercase and lowercase complex and fldSimple SEQ fields plus cached result preservation. * feat(seq-fields): add shared SEQ evaluator Add a pure SequenceFieldEvaluator next to the shared SEQ parser so layout, API, and update paths can share counter semantics without importing editor or rendering internals. The evaluator owns per-identifier counters, heading serial tracking for \s resets, explicit \r reset precedence, conservative field-argument fallback behavior, hidden-result handling, and shared contract-based number formatting. Add focused coverage for the required Phase 3 numbering, reset, repeat-current, hidden, formatting, empty-identifier, field-argument, and initial-counter cases. * feat(seq-fields): resolve SEQ display during layout Add SEQ token metadata to TextRun and emit sequence-field layout tokens without relying on a fake zero placeholder. Resolve SEQ display in a post-assembly toFlowBlocks pass using the shared SequenceFieldEvaluator so paragraph, table, list-compatible, and cache-hit blocks are renumbered in document order before layout measurement. Cover numbering modes, formatting, heading-level restarts, story isolation, table traversal, and FlowBlockCache renumbering with focused layout-adapter tests. * feat(seq-fields): recompute SEQ fields for API updates Add a shared transaction helper that recomputes body SEQ fields in ProseMirror state using the shared parser/evaluator and style-aware heading resolution from the layout adapter when converter style data is available. Wire raw field insertion, field rebuild, caption insertion/configuration, and F9 updates through the helper while preserving existing TOC and document-stat dispatch behavior. Field discovery now reads sequenceField.resolvedNumber as the resolved display text. Cover updater scopes, caption/field wrapper recomputation, field resolver readback, and F9 SEQ refresh behavior with focused tests. * fix(seq-fields): export current SEQ results Update sequenceField export so current evaluated resolvedNumber values are authoritative, including empty current results for hidden fields. Preserve imported cached child content when results are not current, with a fallback to non-current resolvedNumber only when no cached child content exists. Add export-routing tests for current result runs, hidden empty output, cached-content preservation, resolvedNumber fallback, and raw instructionTokens preservation. Add PM-updater coverage documenting Phase 7 conservative field-argument behavior: cached text wins, no-cache references repeat the previous counter, and counters do not advance. * fix(seq-fields): parse attached restart switches Handle Word-emitted SEQ switches such as \r0 and \s1 by normalizing attached numeric values through the existing restart parser path. This fixes the NDA browser-test case where hidden level reset fields imported with restartNumber null, causing the first visible level2 SEQ field to render as 2 instead of 1. Adds parser coverage for attached restart switches and a layout regression for hidden restart-zero fields seeding the next visible value. * fix(seq-fields): recompute F9 after TOC state changes Build the stat/SEQ update transaction from the current editor state after TOC updates dispatch their own transactions. This preserves TOC edits and shifted SEQ positions when F9 updates both TOC and SEQ fields. * fix(seq-fields): detect lowercase caption SEQ fallback Use the shared case-insensitive SEQ instruction check when caption discovery falls back to sequenceField nodes. This keeps imported lowercase seq captions discoverable by the API. * fix(seq-fields): restore legacy format default Keep the sequenceField schema default at ARABIC for backward compatibility while the shared parser continues to default parsed instructions to Arabic. * fix(seq-fields): preserve F9 SEQ selection across TOC edits Remember whether the original F9 selection contained a SEQ field before TOC updates dispatch their own transactions. This lets the fresh post-TOC transaction recompute SEQ fields even when TOC edits shift the selected field position. * refactor(seq-fields): share parsed attr projection Add sequenceFieldAttrsFromParsed beside the SEQ parser and use it from import, raw field insertion, caption insertion/configuration, and PM recompute. This keeps parsed instruction attrs and null/default normalization in one place. * fix(converter): parse attached SEQ format switches * fix(measuring): skip empty text runs during paragraph measurement Empty text runs were flowing into the segment/space accounting, where a trailing run boundary could be counted as a justification space and an oversized empty run's font size could inflate the visible line height. Short-circuit empty runs after recording font state so they no longer produce phantom spaces, segments, or height. * feat(numpages): apply field-switch formatting to total page count fields (SD-3327) (#3637) * feat(numpages): add numeric picture field metadata Extend the shared page-number field format contract with ordinal and numericPicture metadata for later NUMPAGES parser/formatter steps. Add the hidden total-page-number PM attr, expose it on the typed node attrs, and thread it through the layout adapter format mapper with focused coverage. Widen the layout engine page-number state annotation to the shared PageNumberFormat type so the expanded contract remains compile-compatible. * feat(numpages): preserve numeric picture switches Extend parsePageNumberFieldSwitches so Ordinal general switches map to the new ordinal page-number format and non-empty non-zero numeric pictures are retained in pageNumberNumericPicture instead of being discarded. Keep all-zero numeric pictures on the existing zero-padding path for backwards compatibility, and parse switch arguments from the uncollapsed instruction text so quoted picture literals keep their internal whitespace. * feat(numpages): format ordinal and picture counts Extend the shared page-number formatter with English ordinal output and numeric-picture dispatch for preserved NUMPAGES switches. Numeric pictures now take precedence over enum formats and zero padding, while ordinal and existing page-number formats continue through the shared enum formatter. Focused contracts tests cover ordinal suffix rules, numeric-picture dispatch, and page-number normalization. * fix(numpages): format rebuild and shape results Route shape/textbox NUMPAGES through the existing simple pageNumberFormat enum path, matching the SECTIONPAGES branch without adding shape numeric-picture or zero-padding support. Format fields.rebuild total-page-number output with formatPageNumberFieldValue using preserved pageNumberFormat, pageNumberZeroPadding, and pageNumberNumericPicture attrs, and cover rebuilt resolvedText/text content for enum and numeric-picture cases. * fix(numpages): preserve inserted field switches Parse raw NUMPAGES instructions during fields.insert and pass the resulting page-number attrs into total-page-number creation. This preserves numeric picture and general format switches for header/footer NUMPAGES insertion without changing the existing body-insertion restriction. * test(numpages): cover switched field evaluation Add focused coverage for complex and fldSimple NUMPAGES imports, numeric-picture PM/export round-trip, live total-page-count formatting, and shape NUMPAGES formatting. Also thread numeric-picture attrs through the existing export formatter and align the single-block totalPageCount resolver with the formatted runtime path so the new tests exercise the intended behavior. * fix(numpages): preserve F9 field formatting Format total-page-number refreshes in FieldUpdate with the same preserved page-number metadata used by rebuild/export paths. Adds regression coverage for numeric-picture NUMPAGES updates. * fix(numpages): expose preserved field instructions Prefer stored total-page-number instructions during field discovery so fields.list/get report switched NUMPAGES instructions preserved by insert/import. * fix(numpages): format legacy SVG shape totals Apply simple pageNumberFormat handling to NUMPAGES in the legacy SVG text helper, matching the DomPainter shape path and existing PAGE/SECTIONPAGES behavior. * fix(numpages): preserve inserted picture whitespace Parse NUMPAGES insert instructions from the original raw instruction so quoted numeric-picture literals keep significant internal whitespace. * fix(numpages): format edit-mode total pages Apply preserved NUMPAGES formatting in header/footer edit-mode NodeViews and cached DOM refreshes so switched totals match layout/export behavior. * fix(numpages): preserve split header field pictures Join header/footer instrText fragments verbatim so split quoted NUMPAGES numeric pictures do not gain synthetic spaces before parsing. * fix(numpages): preserve quoted picture instructions Keep significant whitespace inside quoted NUMPAGES numeric-picture instructions while still normalizing field-code whitespace outside quotes. * refactor(numpages): share field format attrs mapper Route NUMPAGES formatting call sites through the existing getPageNumberFieldFormat helper and make zeroPadding handling consistent. * test(numpages): cover ordinal and split switch edges Add regression coverage for no-space split instrText switches, ordinal shape NUMPAGES rendering, and three-digit teen ordinal suffixes. * fix(converter): preserve split page field switch args * fix(doc-api): normalize section page numbering format in sections resolver Validate the raw numbering format against the supported SectionPageNumberingFormat values before mapping a section range to a section domain, falling back to undefined for unrecognized formats. Return undefined when no page numbering fields are present so empty numbering objects don't shadow the parsed page numbering.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The
PAGEfield already honored Word formatting switches (\* roman,\# 0000, etc.), butNUMPAGES(total page count) fields rendered as a bare integer everywhere. This branch threads page-number field-switch formatting through every code path that resolves a total-page-count value, so aNUMPAGESfield formatted as Roman numerals, zero-padded, ordinal, or with a numeric picture now renders consistently across pagination, the DOM painter, SVG shapes, headers/footers, F9 field refresh, and the Document API.It also adds two new switch capabilities shared by all page-number fields:
ordinalformat (\* Ordinal→1st,2nd,3rd,11th…).\# "Page 0") preserved asnumericPicturerather than being collapsed to plain zero-padding.Changes
Formatting contract (
layout-engine/contracts)PageNumberFieldFormatgains anordinalformat option and anumericPicturefield.formatPageNumberadds ordinal suffix logic;formatPageNumberFieldValueroutes through the numeric-picture formatter when one is present.Field-switch parsing (
super-converter/field-references)parsePageNumberFieldSwitchesnow recognizesOrdinaland captures non-zero numeric-picture literals aspageNumberNumericPicture(instead of breaking out on the first\#match).normalizeInstructionWhitespacecollapses field-code whitespace outside quoted switch arguments while preserving significant spaces inside numeric-picture literals.preProcessPageFieldsOnlyinserts a switch-boundary space when concatenatinginstrTextfragments (e.g.NUMPAGES+\* Ordinalsplit across runs) so switches aren't glued to preceding tokens.Resolution paths
resolvePageTokens):totalPageCounttokens now format viaformatPageNumberFieldValueand clear their field-format metadata after resolution.activeNumberFormatretyped to the sharedPageNumberFormat.NUMPAGESrendering appliespageNumberFormatwhen present.posAtDOM→nodeAt).field-wrappers,field-resolver):NUMPAGESinsert now parses and stores switches; rebuild formats the fresh value; the synthetic node resolves its preserved instruction.field-updateF9,page-numbernode view): total-page-number nodes format their value with the resolved field format.Types
TotalPageCountAttrs.pageNumberNumericPictureadded;pageNumberFormatnarrowed toPageNumberFormat.f20e00310extracts a shared field-format attrs mapper to avoid duplicating theattrs → formatprojection.Testing
Adds/extends unit tests across all touched layers (
page-number-formatting,resolvePageTokens, DOM painter,HeaderFooterRegistry, switch parsing, whitespace normalization,totalPageNumber-translator,field-resolver,field-wrappers,field-update,page-number,svg-utils) covering ordinal output, numeric pictures, split/quoted switch instructions, and edit-mode/F9 formatting.