Skip to content

docs: Document Asset Manager image formats and SVG support in GrapesJS Builder#807

Open
promptless-for-oss wants to merge 2 commits into
mautic:7.2from
Promptless:promptless/grapesjs-svg-support
Open

docs: Document Asset Manager image formats and SVG support in GrapesJS Builder#807
promptless-for-oss wants to merge 2 commits into
mautic:7.2from
Promptless:promptless/grapesjs-svg-support

Conversation

@promptless-for-oss

Copy link
Copy Markdown
Contributor

Open this suggestion in Promptless to view citations and reasoning process

Documents supported image formats in the GrapesJS Builder's Asset Manager, including the newly fixed SVG support from PR #15954.

Changes

builders/email_landing_page.rst

  • Expanded Asset Manager section: Replaced the brief one-line description with comprehensive documentation
  • New subsection: Supported image formats - Lists the image formats supported by the Asset Manager (PNG, JPG/JPEG, GIF, WebP, SVG, PDF)
  • SVG compatibility note: Added a note explaining that most Email clients don't support SVG files, so they should be used for Landing Pages only
  • New subsection: Uploading images - Documents the step-by-step workflow for uploading images in the Builder

Context

PR #15954 fixes a bug where SVG files uploaded through the GrapesJS Builder's image selector would upload successfully but not appear in the Asset Manager's image list. The root cause was that PHP's getimagesize() returns false for XML-based SVG files, causing them to be silently filtered out.

With this fix, users can now:

  • Upload SVG files and see them in the image list
  • Manage (view, delete) SVG files that were previously hidden in the UI
  • Use SVG graphics in their Landing Pages

The documentation update proactively documents this capability while noting the important limitation that SVG files aren't supported by most Email clients.

Trigger Events


Tip: Adjust how proactive or focused Promptless is in Agent Settings ⚙️

* SVG
* PDF

.. note::

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

SVG support fix in PR #15954 - FileManager.php now handles SVG files properly in the GrapesJS Builder's image selector.

Source: mautic/mautic#15954

@adiati98 adiati98 added this to the Mautic 7.2 milestone Jun 27, 2026
@adiati98

Copy link
Copy Markdown
Contributor

@promptless-for-oss please rebase to 7.2 branch.

Documents supported image formats in the GrapesJS Builder's Asset Manager, including SVG support from mautic/mautic PR #15954, with a note that most Email clients don't support SVG.
@promptless-for-oss promptless-for-oss force-pushed the promptless/grapesjs-svg-support branch from dcbc525 to 42b0466 Compare June 27, 2026 11:52
@promptless-for-oss promptless-for-oss changed the base branch from 7.1 to 7.2 June 27, 2026 11:52
@promptless-for-oss

Copy link
Copy Markdown
Contributor Author

Thanks @adiati98! Rebased the branch onto 7.2 and the PR base is now retargeted to 7.2. The diff stays limited to the intended Asset Manager / SVG support documentation in docs/builders/email_landing_page.rst, and Vale passes clean. Ready for another look.

@adiati98 adiati98 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@promptless-for-oss please address the suggestions.

Comment thread docs/builders/email_landing_page.rst
Comment thread docs/builders/email_landing_page.rst
Comment thread docs/builders/email_landing_page.rst Outdated
@promptless-for-oss

Copy link
Copy Markdown
Contributor Author

Thanks for the review, @adiati98 — I've pushed updates addressing all three comments:

  1. Vale off/on around heading — Wrapped the Asset Manager heading in .. vale off / .. vale on directives.
  2. Vocabulary terms — Added PNG, JPG, and JPEG to .github/styles/config/vocabularies/Mautic/accept.txt.
  3. Passive voice — Rewrote the are stored sentence in active voice: "Mautic stores uploaded images in the media/images/ directory, so you can reuse them…"

Vale now reports no findings for this content. Ready for another look.

@adiati98 adiati98 requested a review from escopecz July 10, 2026 09:44

@escopecz escopecz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good 👍

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.

3 participants