Skip to content

fix: paragraph autospacing calculation (SD-1653)#1877

Merged
harbournick merged 6 commits intomainfrom
luccas/sd-1653-fix-paragraph-autospacing-to-ignore-explicit-values
Jan 30, 2026
Merged

fix: paragraph autospacing calculation (SD-1653)#1877
harbournick merged 6 commits intomainfrom
luccas/sd-1653-fix-paragraph-autospacing-to-ignore-explicit-values

Conversation

@luccas-harbour
Copy link
Contributor

  • Fixes logic behind before/after paragraph spacing when auto spacing is enabled
  • Fixes line height measurement so that it doesn't re-apply default single spacing multiplier on top of value calculated by pm-adapter

@linear
Copy link

linear bot commented Jan 29, 2026

@luccas-harbour luccas-harbour marked this pull request as ready for review January 29, 2026 19:58
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 90fab31780

ℹ️ 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".

Copy link
Contributor

@caio-pizzol caio-pizzol left a comment

Choose a reason for hiding this comment

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

Solid cleanup of line spacing normalization. Left a couple of questions inline:

  • The autospacing calculation may need the same multiplier→twips conversion as normalizeLineValue
  • Wanted to confirm the font-dependent behavior for exact spacing is intentional

LGTM otherwise - the spec-compliant autospacing override and list handling look good.

@caio-pizzol caio-pizzol self-requested a review January 30, 2026 13:39
Copy link
Contributor

@caio-pizzol caio-pizzol left a comment

Choose a reason for hiding this comment

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

LGTM

@caio-pizzol caio-pizzol enabled auto-merge (squash) January 30, 2026 13:39
Copy link
Collaborator

@harbournick harbournick left a comment

Choose a reason for hiding this comment

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

LGTM - nice work!

@harbournick harbournick disabled auto-merge January 30, 2026 19:21
@harbournick harbournick merged commit b7dec3c into main Jan 30, 2026
5 checks passed
@harbournick harbournick deleted the luccas/sd-1653-fix-paragraph-autospacing-to-ignore-explicit-values branch January 30, 2026 19:21
@superdoc-bot
Copy link

superdoc-bot bot commented Jan 30, 2026

🎉 This PR is included in version 1.11.0-next.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@superdoc-bot
Copy link

superdoc-bot bot commented Jan 30, 2026

🎉 This PR is included in version 1.0.1-next.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

superdoc-bot bot pushed a commit that referenced this pull request Feb 3, 2026
# [1.1.0](template-builder-v1.0.0...template-builder-v1.1.0) (2026-02-03)

### Bug Fixes

* cli package public ([7dad84d](7dad84d))
* cli skill uses latest ([b74d9ba](b74d9ba))
* **cli:** document -h flag in help text ([020c4a0](020c4a0))
* **cli:** document -h flag in help text ([68fa42f](68fa42f))
* **cli:** move bundled deps to devDependencies ([6a362ed](6a362ed))
* **cli:** remove bundled deps from package.json ([f90d4af](f90d4af))
* console log ([4b64109](4b64109))
* paragraph autospacing calculation (SD-1653) ([#1877](#1877)) ([b7dec3c](b7dec3c))

### Features

* add esign package to monorepo ([#1895](#1895)) ([532e2b5](532e2b5))
* **cli:** add claude code skill ([#1903](#1903)) ([e55ce56](e55ce56))
* new cli app ([#1902](#1902)) ([0c6aeb0](0c6aeb0))
* **template-builder:** add cspNonce support ([#1911](#1911)) ([5b7b34e](5b7b34e))
@superdoc-bot
Copy link

superdoc-bot bot commented Feb 3, 2026

🎉 This PR is included in version 1.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

harbournick pushed a commit that referenced this pull request Feb 6, 2026
# [1.11.0](v1.10.0...v1.11.0) (2026-02-06)

### Bug Fixes

* cli package public ([7dad84d](7dad84d))
* cli skill uses latest ([b74d9ba](b74d9ba))
* **cli:** document -h flag in help text ([020c4a0](020c4a0))
* **cli:** document -h flag in help text ([68fa42f](68fa42f))
* **cli:** move bundled deps to devDependencies ([6a362ed](6a362ed))
* **cli:** remove bundled deps from package.json ([f90d4af](f90d4af))
* console log ([4b64109](4b64109))
* document dropdown resets ([#1883](#1883)) ([b552d2e](b552d2e))
* lock file ([1edf741](1edf741))
* make Ctrl-a select all content, and fix select all when there are tables in doc ([#1886](#1886)) ([c87c1ab](c87c1ab))
* package lock ([11f47fa](11f47fa))
* paragraph autospacing calculation (SD-1653) ([#1877](#1877)) ([b7dec3c](b7dec3c))
* run color overwrite in applyInlineRunProperties (SD-1585) ([#1885](#1885)) ([6051dd4](6051dd4))
* selection across pages with drag ([#1884](#1884)) ([bb44999](bb44999))
* **super-converter:** handle ECMA-376 percentage strings in table widths (SD-1633) ([#1844](#1844)) ([fb9fd52](fb9fd52))
* **super-converter:** return transparent instead of blue for shapes ([#1854](#1854)) ([6ac29d7](6ac29d7)), closes [#5b9bd5](https://github.com/superdoc-dev/superdoc/issues/5b9bd5)

### Features

* add esign package to monorepo ([#1895](#1895)) ([532e2b5](532e2b5))
* add template-builder package to monorepo ([#1888](#1888)) ([0aec624](0aec624))
* **cli:** add claude code skill ([#1903](#1903)) ([e55ce56](e55ce56))
* new cli app ([#1902](#1902)) ([0c6aeb0](0c6aeb0))
* **template-builder:** add cspNonce support ([#1911](#1911)) ([5b7b34e](5b7b34e))
* **vscode-ext:** sync from main ([47c50f7](47c50f7))
* **vscode-ext:** sync from main ([226bcf1](226bcf1))
* **vscode-ext:** sync from main ([eced1f3](eced1f3))
* **vscode-ext:** sync from main ([36b48a6](36b48a6))
* **vscode-ext:** sync from main ([cdc00df](cdc00df))
@harbournick
Copy link
Collaborator

🎉 This PR is included in superdoc v1.11.0

The release is available on GitHub release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants