From fbf8528814cb4cc6305e4c2995c387d1c046e42e Mon Sep 17 00:00:00 2001 From: Kris Date: Mon, 30 Oct 2023 10:55:39 -0400 Subject: [PATCH] UX: set tag sort based on siteSetting (#24147) --- .../javascripts/discourse/app/controllers/tags-index.js | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/app/assets/javascripts/discourse/app/controllers/tags-index.js b/app/assets/javascripts/discourse/app/controllers/tags-index.js index c42122d5f8b..b7615c13185 100644 --- a/app/assets/javascripts/discourse/app/controllers/tags-index.js +++ b/app/assets/javascripts/discourse/app/controllers/tags-index.js @@ -13,6 +13,7 @@ export default Controller.extend({ modal: service(), sortedByCount: true, sortedByName: false, + sortAlphabetically: alias("siteSettings.tags_sort_alphabetically"), canAdminTags: alias("currentUser.staff"), groupedByCategory: notEmpty("model.extras.categories"), groupedByTagGroup: notEmpty("model.extras.tag_groups"), @@ -20,7 +21,13 @@ export default Controller.extend({ init() { this._super(...arguments); - this.sortProperties = ["totalCount:desc", "id"]; + const isAlphaSort = this.sortAlphabetically; + + this.setProperties({ + sortedByCount: isAlphaSort ? false : true, + sortedByName: isAlphaSort ? true : false, + sortProperties: isAlphaSort ? ["id"] : ["totalCount:desc", "id"], + }); }, @discourseComputed("groupedByCategory", "groupedByTagGroup")