From 782da448a26bd5f84336afc3a9d3d2e242c787a8 Mon Sep 17 00:00:00 2001 From: Guo Xiang Tan Date: Thu, 6 Jun 2019 16:45:12 +0800 Subject: [PATCH] FIX: Missing `title` attribute when quoting an image. --- app/assets/javascripts/discourse/lib/to-markdown.js.es6 | 3 ++- test/javascripts/lib/to-markdown-test.js.es6 | 3 +++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/app/assets/javascripts/discourse/lib/to-markdown.js.es6 b/app/assets/javascripts/discourse/lib/to-markdown.js.es6 index 8e4eaa4883..b4fc915023 100644 --- a/app/assets/javascripts/discourse/lib/to-markdown.js.es6 +++ b/app/assets/javascripts/discourse/lib/to-markdown.js.es6 @@ -241,6 +241,7 @@ export class Tag { let alt = attr.alt || pAttr.alt || ""; const width = attr.width || pAttr.width; const height = attr.height || pAttr.height; + const title = attr.title; if (width && height) { const pipe = this.element.parentNames.includes("table") @@ -249,7 +250,7 @@ export class Tag { alt = `${alt}${pipe}${width}x${height}`; } - return "![" + alt + "](" + src + ")"; + return `![${alt}](${src}${title ? ` "${title}"` : ""})`; } return ""; diff --git a/test/javascripts/lib/to-markdown-test.js.es6 b/test/javascripts/lib/to-markdown-test.js.es6 index f63a004c26..2cb28ba25f 100644 --- a/test/javascripts/lib/to-markdown-test.js.es6 +++ b/test/javascripts/lib/to-markdown-test.js.es6 @@ -166,6 +166,9 @@ QUnit.test("converts img tag", assert => { let html = ``; assert.equal(toMarkdown(html), `![|100x50](${url})`); + html = ``; + assert.equal(toMarkdown(html), `![|100x50](${url} "some title")`); + html = `
description
`; assert.equal(toMarkdown(html), `![description|50x100](${url})`);