Skip to content

fix(pptx): follow slide links outside slide shows - #38

Merged
ChristopherVR merged 1 commit into
ChristopherVR:mainfrom
yunfeizhu:main
Oct 9, 2026
Merged

ChristopherVR merged 1 commit into
ChristopherVR:mainfrom
yunfeizhu:main

Conversation

@yunfeizhu

@yunfeizhu yunfeizhu commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

Internal text-run and shape slide links now navigate inside normal and read-only viewers. Text-run links also use the slide-show action runner instead of opening slideN.xml. All five PPTX bindings expose a cancellable host callback and hide read-only authoring badges. Editing keeps Ctrl/Cmd+Click navigation.

Fixes #37.

Validation: the reporter's deck and 110 local browser cases passed across all five bindings; builds, typechecks and clean-install package checks passed.

Full fork CI completed: PPTX build/checks and 50/51 browser jobs passed, including all 35 new slide-link regressions. React's two table-markup failures reproduce on unmodified upstream. Other failures are formatting in unchanged Visio documentation and a Svelte collaboration pixel-delta assertion: its trace shows identical synchronized geometry while the peer's zoom changes. The collaboration case passes locally on both upstream and this commit.

@github-actions github-actions Bot added documentation Improvements or additions to documentation tests Test-only change area: ui ooxml-ui custom elements viewer: pptx PowerPoint viewer, demos and tests labels Oct 9, 2026
@ChristopherVR
ChristopherVR merged commit f3e99cb into ChristopherVR:main Oct 9, 2026
79 of 82 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ui ooxml-ui custom elements documentation Improvements or additions to documentation tests Test-only change viewer: pptx PowerPoint viewer, demos and tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Slide-jump links don't navigate outside slide show; text-run links open slideN.xml in a new tab

2 participants