fix(modal): title extraction method (jquery removed .context)

[Gomez]

Fixes JSError: Uncaught TypeError: can't access property "innerText", e.$el.context is undefined if data-pat-plone-modal="image: true"

[Copilot]

Pull request overview

This PR fixes a runtime error in the modal pattern’s image-modal path by removing reliance on the deprecated/removed jQuery .context property when extracting the modal title.

Changes:

• Replace self.$el.context.innerText title extraction with self.$el.text() to avoid context is undefined errors for image: true.

Comments suppressed due to low confidence (1)

src/pat/modal/modal.js:660

• title, src, and srcset are concatenated into an HTML string. If any of these values contain characters like < or quotes (e.g., via encoded text or programmatically-set attributes), this can result in HTML injection. Prefer constructing the DOM with jQuery and setting text/attributes via .text() / .attr() to ensure proper escaping.

        var title = $.trim(self.$el.text()) || "Image";
        // XXX aria?
        self.$raw = $(
            "<div><h1>" +
            title +
            '</h1><div id="content"><div class="modal-image"><img src="' +

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[petschki]

LGTM. Thanks!

[petschki]

@Gomez would be great if you could cherry pick this commit and create a second PR based on master.