Skip to content

fix(super-converter): return transparent instead of blue for shapes#1854

Merged
harbournick merged 3 commits intomainfrom
caio/sd-1651-fix-text-box-fill-defaulting-to-blue-instead-of-transparent
Jan 30, 2026
Merged

fix(super-converter): return transparent instead of blue for shapes#1854
harbournick merged 3 commits intomainfrom
caio/sd-1651-fix-text-box-fill-defaulting-to-blue-instead-of-transparent

Conversation

@caio-pizzol
Copy link
Contributor

@caio-pizzol caio-pizzol commented Jan 27, 2026

image

…ithout fill

SD-1651: When a text box or shape has no explicit fill color specified in
spPr and no style element, extractFillColor was returning the default
blue (#5b9bd5) instead of null (transparent).

Per ECMA-376 spec, shapes without fill specification should be transparent.
This fix changes three return statements from returning blue to returning
null when:
- No style element exists
- Style exists but no fillRef
- fillRef exists but no schemeClr

This fixes text boxes appearing with blue backgrounds when they should be
transparent.
@linear
Copy link

linear bot commented Jan 27, 2026

@caio-pizzol caio-pizzol requested review from artem-harbour and harbournick and removed request for harbournick January 27, 2026 18:27
@caio-pizzol caio-pizzol self-assigned this Jan 27, 2026
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

@harbournick harbournick merged commit 6ac29d7 into main Jan 30, 2026
4 of 5 checks passed
@harbournick harbournick deleted the caio/sd-1651-fix-text-box-fill-defaulting-to-blue-instead-of-transparent branch January 30, 2026 00:24
harbournick pushed a commit that referenced this pull request Jan 30, 2026
## [1.10.1-next.1](v1.10.0...v1.10.1-next.1) (2026-01-30)

### Bug Fixes

* **super-converter:** return transparent instead of blue for shapes ([#1854](#1854)) ([6ac29d7](6ac29d7)), closes [#5b9bd5](https://github.com/superdoc-dev/superdoc/issues/5b9bd5)
@harbournick
Copy link
Collaborator

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

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.

2 participants