Skip to content

Dev3 expand anim+text wrapfix (#261+#267) - #269

Open
hiroshisuga wants to merge 35 commits into
v3.0.x-releasefrom
dev3_expandAnim+textWrapfix
Open

Dev3 expand anim+text wrapfix (#261+#267)#269
hiroshisuga wants to merge 35 commits into
v3.0.x-releasefrom
dev3_expandAnim+textWrapfix

Conversation

@hiroshisuga

@hiroshisuga hiroshisuga commented Jul 18, 2026

Copy link
Copy Markdown
Owner

when you merge this to #240, be careful about presentation-uploader/component.jsx.
At 'renderPresentationItem', you only need to add the two lines in #240 to THIS code.

@hiroshisuga hiroshisuga changed the title Dev3 expand anim+text wrapfix Dev3 expand anim+text wrapfix (#261+#267) Jul 18, 2026
Text emergence animation was not properly processed before this fix.

PowerPoint stores “With Previous” and “After Previous” animations as separate timing nodes, even though they do not require an additional click. The previous implementation treated these nodes as separate click groups, which caused extra intermediate slides to be generated.

This change associates withEffect and afterEffect actions with the most recent clickEffect group. As a result, all animations triggered by the same click are expanded into a single static slide state.
@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

🚨 Automated tests failed

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