From 2b3e248927076732d20204537a0c7fbbcf7d4bcf Mon Sep 17 00:00:00 2001 From: Sam Saffron Date: Tue, 5 Nov 2019 16:15:27 +1100 Subject: [PATCH] FIX: on mobile setting active on navItem would not work navigation bar would not have the current nav item selected when it overrode --- .../discourse/components/navigation-bar.js.es6 | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/app/assets/javascripts/discourse/components/navigation-bar.js.es6 b/app/assets/javascripts/discourse/components/navigation-bar.js.es6 index bf2947fc1d..eb8cb454b3 100644 --- a/app/assets/javascripts/discourse/components/navigation-bar.js.es6 +++ b/app/assets/javascripts/discourse/components/navigation-bar.js.es6 @@ -22,9 +22,11 @@ export default Component.extend({ if (filterMode.indexOf("top/") === 0) { filterMode = "top"; } - var item = navItems.find( - i => i.get("filterMode").indexOf(filterMode) === 0 - ); + let item = navItems.find(i => i.active === true); + + item = + item || navItems.find(i => i.get("filterMode").indexOf(filterMode) === 0); + if (!item) { let connectors = this.connectors; let category = this.category;