From 1a731dcff1bc83c58b276775438ae6c13ca2cd4e Mon Sep 17 00:00:00 2001
From: Joffrey JAFFEUX <j.jaffeux@gmail.com>
Date: Mon, 19 Aug 2019 09:33:54 +0200
Subject: [PATCH] FIX: prevents mobile nav to create an error when
 unregistering click events (#8016)

This what happening when clickin on "All groups", as it was trying to get element of a destroying component.
---
 .../discourse/components/mobile-nav.js.es6          | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/app/assets/javascripts/discourse/components/mobile-nav.js.es6 b/app/assets/javascripts/discourse/components/mobile-nav.js.es6
index bba0162b81..49392fc336 100644
--- a/app/assets/javascripts/discourse/components/mobile-nav.js.es6
+++ b/app/assets/javascripts/discourse/components/mobile-nav.js.es6
@@ -4,7 +4,7 @@ export default Ember.Component.extend({
   @on("init")
   _init() {
     if (!this.get("site.mobileView")) {
-      var classes = this.desktopClass;
+      let classes = this.desktopClass;
       if (classes) {
         classes = classes.split(" ");
         this.set("classNames", classes);
@@ -31,6 +31,8 @@ export default Ember.Component.extend({
   },
 
   didInsertElement() {
+    this._super(...arguments);
+
     this._updateSelectedHtml();
   },
 
@@ -43,9 +45,12 @@ export default Ember.Component.extend({
           $(window)
             .off("click.mobile-nav")
             .on("click.mobile-nav", e => {
-              let expander = $(this.element.querySelector(".expander"));
-              expander = expander && expander[0];
-              if ($(e.target)[0] !== expander) {
+              if (!this.element || this.isDestroying || this.isDestroyed) {
+                return;
+              }
+
+              const expander = this.element.querySelector(".expander");
+              if (expander && e.target !== expander) {
                 this.set("expanded", false);
                 $(window).off("click.mobile-nav");
               }