github-mirror/discourse
2
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2024-12-14 05:33:54 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

FIX: throttles topic tracking shortcut and enforces topic id (#9159)

Browse Source
This commit is contained in:
Joffrey JAFFEUX 2020-03-10 17:17:54 +01:00 committed by jjaffeux
parent b4b0443d63
commit c4cd864c26
2 changed files with 21 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
app/assets/javascripts/discourse/lib/keyboard-shortcuts.js.es6
View File

@ -4,6 +4,7 @@ import DiscourseURL from "discourse/lib/url";
import Composer from "discourse/models/composer";
import { minimumOffset } from "discourse/lib/offset-calculator";
import { ajax } from "discourse/lib/ajax";
import { throttle } from "@ember/runloop";
const bindings = {
"!": { postAction: "showFlags" },
@ -299,34 +300,26 @@ export default {
},
setTrackingToMuted(event) {
this.appEvents.trigger("topic-notifications-button:changed", {
type: "notification",
id: 0,
event
});
throttle(this, "_setTracking", { id: 0, event }, 250, true);
},
setTrackingToRegular(event) {
this.appEvents.trigger("topic-notifications-button:changed", {
type: "notification",
id: 1,
event
});
throttle(this, "_setTracking", { id: 1, event }, 250, true);
},
setTrackingToTracking(event) {
this.appEvents.trigger("topic-notifications-button:changed", {
type: "notification",
id: 2,
event
});
throttle(this, "_setTracking", { id: 2, event }, 250, true);
},
setTrackingToWatching(event) {
throttle(this, "_setTracking", { id: 3, event }, 250, true);
},
_setTracking(params) {
this.appEvents.trigger("topic-notifications-button:changed", {
type: "notification",
id: 3,
event
id: params.id,
event: params.event
});
},

11
app/assets/javascripts/select-kit/components/topic-notifications-button.js.es6
View File

@ -28,6 +28,17 @@ export default Component.extend({
},
_changeTopicNotificationLevel(level) {
// this change is coming from a keyboard event
if (level.event) {
const topicSectionNode = level.event.target.querySelector("#topic");
if (topicSectionNode && topicSectionNode.dataset.topicId) {
const topicId = parseInt(topicSectionNode.dataset.topicId, 10);
if (topicId && topicId !== this.topic.id) {
return;
}
}
}
if (level.id !== this.notificationLevel) {
this.topic.details.updateNotifications(level.id);
}