From 9fb5bfd93d025216308c3bcb4d637a8a67a2b7e4 Mon Sep 17 00:00:00 2001 From: Krzysztof Kotlarek Date: Fri, 24 Feb 2023 16:11:45 +1100 Subject: [PATCH] FIX: use category description for sidebar link title (#20436) Value in `description_text` is escaped. We would like to display `/` properly in link title. --- .../sidebar/user/categories-section/category-section-link.js | 2 +- .../tests/acceptance/sidebar-user-categories-section-test.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/app/assets/javascripts/discourse/app/lib/sidebar/user/categories-section/category-section-link.js b/app/assets/javascripts/discourse/app/lib/sidebar/user/categories-section/category-section-link.js index 1427b50bdf..c76a314caa 100644 --- a/app/assets/javascripts/discourse/app/lib/sidebar/user/categories-section/category-section-link.js +++ b/app/assets/javascripts/discourse/app/lib/sidebar/user/categories-section/category-section-link.js @@ -131,7 +131,7 @@ export default class CategorySectionLink { } get title() { - return this.category.description_text; + return this.category.description; } get text() { diff --git a/app/assets/javascripts/discourse/tests/acceptance/sidebar-user-categories-section-test.js b/app/assets/javascripts/discourse/tests/acceptance/sidebar-user-categories-section-test.js index 704cf303a3..053df5081d 100644 --- a/app/assets/javascripts/discourse/tests/acceptance/sidebar-user-categories-section-test.js +++ b/app/assets/javascripts/discourse/tests/acceptance/sidebar-user-categories-section-test.js @@ -615,7 +615,7 @@ acceptance("Sidebar - Logged on user - Categories Section", function (needs) { assert.strictEqual( query(`.sidebar-section-link[data-category-id="${category.id}"]`).title, - category.description_text, + category.description, "category description without HTML entity is used as the link's title" ); });