UX: removes silence from chat message actions (#19282)
This commit is contained in:
parent
b83d9fa802
commit
fca6805aca
|
@ -213,14 +213,6 @@ export default Component.extend({
|
|||
});
|
||||
}
|
||||
|
||||
if (this.showSilenceButton) {
|
||||
buttons.push({
|
||||
id: "silence",
|
||||
name: I18n.t("chat.silence"),
|
||||
icon: "microphone-slash",
|
||||
});
|
||||
}
|
||||
|
||||
if (this.showDeleteButton) {
|
||||
buttons.push({
|
||||
id: "deleteMessage",
|
||||
|
@ -256,7 +248,6 @@ export default Component.extend({
|
|||
edit: this.edit,
|
||||
selectMessage: this.selectMessage,
|
||||
flag: this.flag,
|
||||
silence: this.silence,
|
||||
deleteMessage: this.deleteMessage,
|
||||
restore: this.restore,
|
||||
rebakeMessage: this.rebakeMessage,
|
||||
|
@ -393,15 +384,6 @@ export default Component.extend({
|
|||
);
|
||||
},
|
||||
|
||||
@discourseComputed("message")
|
||||
showSilenceButton(message) {
|
||||
return (
|
||||
this.currentUser?.staff &&
|
||||
this.currentUser?.id !== message.user?.id &&
|
||||
!message.chat_webhook_event
|
||||
);
|
||||
},
|
||||
|
||||
@discourseComputed("message")
|
||||
canManageDeletion(message) {
|
||||
return this.currentUser?.id === message.user?.id
|
||||
|
@ -693,11 +675,6 @@ export default Component.extend({
|
|||
}
|
||||
},
|
||||
|
||||
@action
|
||||
silence() {
|
||||
this.adminTools.showSilenceModal(EmberObject.create(this.message.user));
|
||||
},
|
||||
|
||||
@action
|
||||
expand() {
|
||||
this.message.set("expanded", true);
|
||||
|
|
|
@ -236,21 +236,6 @@ acceptance("Discourse Chat - without unread", function (needs) {
|
|||
currentUserDropdown.rowByValue("rebakeMessage").exists(),
|
||||
"it shows the rebake button"
|
||||
);
|
||||
|
||||
assert.notOk(
|
||||
currentUserDropdown.rowByValue("silence").exists(),
|
||||
"it hides the silence button"
|
||||
);
|
||||
|
||||
const notCurrentUserDropdown = selectKit(
|
||||
".chat-message-actions-container[data-id='175'] .more-buttons"
|
||||
);
|
||||
await triggerEvent(".chat-message-container[data-id='175']", "mouseenter");
|
||||
await notCurrentUserDropdown.expand();
|
||||
assert.ok(
|
||||
notCurrentUserDropdown.rowByValue("silence").exists(),
|
||||
"it shows the silence button"
|
||||
);
|
||||
});
|
||||
|
||||
test("Message controls are present and correct for permissions", async function (assert) {
|
||||
|
@ -288,11 +273,6 @@ acceptance("Discourse Chat - without unread", function (needs) {
|
|||
"it hides the flag button"
|
||||
);
|
||||
|
||||
assert.notOk(
|
||||
currentUserDropdown.rowByValue("silence").exists(),
|
||||
"it hides the silence button"
|
||||
);
|
||||
|
||||
assert.ok(
|
||||
currentUserDropdown.rowByValue("deleteMessage").exists(),
|
||||
"it shows the delete button"
|
||||
|
|
Loading…
Reference in New Issue