From 2faa4c2f5fe2937382c10caa5a711499cce6f992 Mon Sep 17 00:00:00 2001 From: Robin Ward Date: Fri, 9 Feb 2018 16:58:04 -0500 Subject: [PATCH] FIX: Don't show personal messages if disabled as a composer action --- .../select-kit/components/composer-actions.js.es6 | 15 +++++++++++++-- config/locales/client.en.yml | 2 ++ 2 files changed, 15 insertions(+), 2 deletions(-) diff --git a/app/assets/javascripts/select-kit/components/composer-actions.js.es6 b/app/assets/javascripts/select-kit/components/composer-actions.js.es6 index 5c0fc2277f6..44c58379744 100644 --- a/app/assets/javascripts/select-kit/components/composer-actions.js.es6 +++ b/app/assets/javascripts/select-kit/components/composer-actions.js.es6 @@ -1,5 +1,5 @@ import DropdownSelectBoxComponent from "select-kit/components/dropdown-select-box"; -import { default as computed } from "ember-addons/ember-computed-decorators"; +import computed from "ember-addons/ember-computed-decorators"; import { default as Composer, PRIVATE_MESSAGE, CREATE_TOPIC, REPLY, EDIT } from "discourse/models/composer"; // Component can get destroyed and lose state @@ -79,7 +79,7 @@ export default DropdownSelectBoxComponent.extend({ }); } - if (action !== PRIVATE_MESSAGE) { + if (this.siteSettings.enable_personal_messages && action !== PRIVATE_MESSAGE) { items.push({ name: I18n.t("composer.composer_actions.reply_as_private_message.label"), description: I18n.t("composer.composer_actions.reply_as_private_message.desc"), @@ -107,6 +107,17 @@ export default DropdownSelectBoxComponent.extend({ }); } + // Edge case: If personal messages are disabled, it is possible to have + // no items which stil renders a button that pops up nothing. In this + // case, add an option for what you're currently doing. + if (action === CREATE_TOPIC && items.length === 0) { + items.push({ + name: I18n.t("composer.composer_actions.create_topic.label"), + description: I18n.t("composer.composer_actions.reply_as_new_topic.desc"), + icon: "mail-forward", + id: "create_topic" + }); + } return items; }, diff --git a/config/locales/client.en.yml b/config/locales/client.en.yml index e95cd09c447..46f59e97716 100644 --- a/config/locales/client.en.yml +++ b/config/locales/client.en.yml @@ -1307,6 +1307,8 @@ en: toggle_whisper: label: Toggle whipser desc: Whispers will only be visible by staff members + create_topic: + label: "New Topic" notifications: tooltip: