Skip to content

Bard Transformer: Recursively map Prosemirror nodes in order to handle deeply nested images#85

Merged
duncanmcclean merged 5 commits intostatamic:mainfrom
tdondich:bugfix/84
Jan 29, 2025
Merged

Bard Transformer: Recursively map Prosemirror nodes in order to handle deeply nested images#85
duncanmcclean merged 5 commits intostatamic:mainfrom
tdondich:bugfix/84

Conversation

@tdondich
Copy link
Copy Markdown
Contributor

@tdondich tdondich commented Jan 25, 2025

fixes #84

Utilizes recursion to navigate through the prosemirror structure to identify any deeply nested image elements and perform asset transformation as needed.

…o perform asset transformation. Add appropriate test.
@tdondich tdondich changed the title Fixes:84 - Add ability to recursively navigate html content to handle asset transformation in deeply nested img elements. fixes #84 - Add ability to recursively navigate html content to handle asset transformation in deeply nested img elements. Jan 25, 2025
@duncanmcclean duncanmcclean changed the title fixes #84 - Add ability to recursively navigate html content to handle asset transformation in deeply nested img elements. Bard Transformer: Recursively map Prosemirror nodes in order to handle deeply nested images Jan 29, 2025
Copy link
Copy Markdown
Member

@duncanmcclean duncanmcclean left a comment

Choose a reason for hiding this comment

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

Thank you!

@duncanmcclean duncanmcclean merged commit e41a14d into statamic:main Jan 29, 2025
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.

BardTransformer fails to handle deep child image elements when performing asset transforming

2 participants