Skip to content

fix: drop phantom slideMaster overrides & correct jpg MIME (#1444)#2

Merged
MBuelowius merged 1 commit into
masterfrom
fix/phantom-slidemaster-1462
Jun 24, 2026
Merged

fix: drop phantom slideMaster overrides & correct jpg MIME (#1444)#2
MBuelowius merged 1 commit into
masterfrom
fix/phantom-slidemaster-1462

Conversation

@MBuelowius

@MBuelowius MBuelowius commented Jun 23, 2026

Copy link
Copy Markdown
Collaborator

Original author: @IgnasStark — adopted from upstream gitbrent/PptxGenJS#1462 (MIT). Credited via Co-Authored-By trailer.

Only slideMaster1.xml is ever written, but [Content_Types].xml emitted one <Override> per slide (slideMaster1..N), referencing parts that do not exist — PowerPoint shows the repair dialog on every multi-slide deck. Now emits a single slideMaster1 override. Also corrects the invalid image/jpg default content type to image/jpeg.

tsc --noEmit clean. dist/ not rebuilt here.

…1444)

Adopts upstream PR gitbrent#1462. In makeXmlContTypes:

- Only slideMaster1.xml is ever written, but [Content_Types].xml emitted one
  <Override> per slide (slideMaster1..N), referencing parts that don't exist.
  PowerPoint flags the dangling refs and shows the repair dialog on every
  multi-slide deck. Now emit a single slideMaster1 override.
- jpg default content type was the invalid "image/jpg"; corrected to "image/jpeg".

Original work by @IgnasStark in gitbrent#1462 (MIT).

Co-Authored-By: IgnasStark <IgnasStark@users.noreply.github.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@MBuelowius
MBuelowius force-pushed the fix/phantom-slidemaster-1462 branch from e1910dc to c953620 Compare June 23, 2026 13:44
@MBuelowius
MBuelowius merged commit 5aa1ee1 into master Jun 24, 2026
@MBuelowius
MBuelowius deleted the fix/phantom-slidemaster-1462 branch June 24, 2026 10:35
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.

1 participant