Skip to content

Commit

Permalink
Merge pull request #1304 from plone/fix-external-images-having-pictur…
Browse files Browse the repository at this point in the history
…evariants-61

fix: Do not add picture-variants classes and attributes to external i…
  • Loading branch information
thet committed Apr 12, 2023
2 parents f9917b1 + a6e6161 commit 8bff993
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions src/pat/tinymce/js/links.js
Original file line number Diff line number Diff line change
Expand Up @@ -664,8 +664,10 @@ export default Base.extend({
var cssclasses = [
"image-richtext",
self.$align.val(),
"picture-variant-" + self.$scale.val(),
];
if(self.linkType !== "externalImage"){
cssclasses.push("picture-variant-" + self.$scale.val())
}
if (captionFromDescription || caption) {
cssclasses.push("captioned");
}
Expand All @@ -681,11 +683,14 @@ export default Base.extend({
"alt": self.$alt.val(),
"class": cssclasses.join(" "),
"data-linkType": self.linkType,
"data-picturevariant": self.$scale.val(),
"data-scale": self.getScaleFromSrcset(self.$scale.val()),
},
self.linkTypes[self.linkType].attributes()
);
if(self.linkType !== "externalImage"){
data["data-picturevariant"] = self.$scale.val();
}

if (caption && !captionFromDescription) {
data["data-captiontext"] = caption;
}
Expand Down

0 comments on commit 8bff993

Please sign in to comment.