FIX: ensures shortcuts work correctly with topic-notifications-button (#8956)

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

@@ -6,11 +6,35 @@ export default Component.extend({
   appendReason: true,
   showFullTitle: true,
 
+  didInsertElement() {
+    this._super(...arguments);
+
+    this.appEvents.on(
+      "topic-notifications-button:changed",
+      this,
+      "_changeTopicNotificationLevel"
+    );
+  },
+
+  willDestroyElement() {
+    this._super(...arguments);
+
+    this.appEvents.off(
+      "topic-notifications-button:changed",
+      this,
+      "_changeTopicNotificationLevel"
+    );
+  },
+
+  _changeTopicNotificationLevel(level) {
+    if (level.id !== this.notificationLevel) {
+      this.topic.details.updateNotifications(level.id);
+    }
+  },
+
   actions: {
-    changeTopicNotificationLevel(newNotificationLevel) {
+    changeTopicNotificationLevel(level, notification) {
-      if (newNotificationLevel !== this.notificationLevel) {
+      this._changeTopicNotificationLevel(notification);
-        this.topic.details.updateNotifications(newNotificationLevel);
-      }
     }
   }
 });

app/assets/javascripts/select-kit/components/topic-notifications-options.js.es6

@@ -16,17 +16,5 @@ export default NotificationsButtonComponent.extend({
 
   i18nPostfix: computed("topic.archetype", function() {
     return this.topic.archetype === "private_message" ? "_pm" : "";
-  }),
+  })
-
-  didInsertElement() {
-    this._super(...arguments);
-
-    this.appEvents.on("topic-notifications-button:changed", this, "onSelect");
-  },
-
-  willDestroyElement() {
-    this._super(...arguments);
-
-    this.appEvents.off("topic-notifications-button:changed", this, "onSelect");
-  }
 });