UX: Better emoji escaping for topic title (#7218)

* FIX: Fixed failing discourse-prometheus-alert-receiver plugin specs
This commit is contained in:
Tim Lange
2019-03-21 09:11:33 +01:00
committed by Régis Hanol
parent 6450969c7b
commit f7b156ffbd
7 changed files with 74 additions and 26 deletions

View File

@ -184,6 +184,23 @@ QUnit.test("Updating the topic title with emojis", async assert => {
);
});
QUnit.test("Updating the topic title with unicode emojis", async assert => {
await visit("/t/internationalization-localization/280");
await click("#topic-title .d-icon-pencil-alt");
await fillIn("#edit-title", "emojis title 👨‍🌾");
await click("#topic-title .submit-edit");
assert.equal(
find(".fancy-title")
.html()
.trim(),
`emojis title <img src="/images/emoji/emoji_one/man_farmer.png?v=${v}" title="man_farmer" alt="man_farmer" class="emoji">`,
"it displays the new title with escaped unicode emojis"
);
});
acceptance("Topic featured links", {
loggedIn: true,
settings: {