From 21c2b32dce63f67e3eb1ecaa960624d62332b1f0 Mon Sep 17 00:00:00 2001
From: Tasso Evangelista <tasso.evangelista@rocket.chat>
Date: Sat, 27 Apr 2019 15:45:08 -0300
Subject: [PATCH] Keep sidenav for admin pages in embedded layout

---
 app/ui-admin/client/adminFlex.html |  8 +++-----
 app/ui-admin/client/adminFlex.js   | 19 +++++++++++--------
 app/ui-master/client/main.js       |  4 +---
 app/ui-utils/client/lib/SideNav.js | 16 +++++-----------
 client/routes/router.js            |  3 +--
 5 files changed, 21 insertions(+), 29 deletions(-)

diff --git a/app/ui-admin/client/adminFlex.html b/app/ui-admin/client/adminFlex.html
index cb8b16ff24ff..0ab4fb2f7739 100644
--- a/app/ui-admin/client/adminFlex.html
+++ b/app/ui-admin/client/adminFlex.html
@@ -2,11 +2,9 @@
 	<aside class="sidebar-light sidebar--medium" role="navigation">
 		<header class="sidebar-flex__header">
 			<h1 class="sidebar-flex__title">{{_ "Administration"}}</h1>
-			{{#unless embeddedVersion}}
-				<button class="sidebar-flex__close-button" data-action="close">
-					{{> icon block="sidebar-flex__close-icon" icon="cross"}}
-				</button>
-			{{/unless}}
+			<button class="sidebar-flex__close-button" data-action="close">
+				{{> icon block="sidebar-flex__close-icon" icon="cross"}}
+			</button>
 		</header>
 		<div class="rooms-list {{#if embeddedVersion}}rooms-list--embedded{{/if}}" aria-label="{{_ "Administration"}}">
 			<ul class="rooms-list__list">
diff --git a/app/ui-admin/client/adminFlex.js b/app/ui-admin/client/adminFlex.js
index ad1e3912ca48..b02ad2f87178 100644
--- a/app/ui-admin/client/adminFlex.js
+++ b/app/ui-admin/client/adminFlex.js
@@ -1,3 +1,5 @@
+import _ from 'underscore';
+import s from 'underscore.string';
 import { ReactiveVar } from 'meteor/reactive-var';
 import { Template } from 'meteor/templating';
 import { TAPi18n } from 'meteor/tap:i18n';
@@ -5,10 +7,10 @@ import { settings } from '../../settings';
 import { CachedCollection } from '../../ui-cached-collection';
 import { SideNav, AdminBox, Layout } from '../../ui-utils';
 import { t } from '../../utils';
-import _ from 'underscore';
-import s from 'underscore.string';
+import { menu } from '../../ui-utils/client';
 
 Template.adminFlex.onCreated(function() {
+	this.isEmbedded = Layout.isEmbedded();
 	this.settingsFilter = new ReactiveVar('');
 	if (settings.cachedCollectionPrivate == null) {
 		settings.cachedCollectionPrivate = new CachedCollection({
@@ -25,10 +27,6 @@ const label = function() {
 	return TAPi18n.__(this.i18nLabel || this._id);
 };
 
-// Template.adminFlex.onRendered(function() {
-// 	$(this.find('.rooms-list')).perfectScrollbar();
-// });
-
 Template.adminFlex.helpers({
 	groups() {
 		const filter = Template.instance().settingsFilter.get();
@@ -77,12 +75,17 @@ Template.adminFlex.helpers({
 		};
 	},
 	embeddedVersion() {
-		return Layout.isEmbedded();
+		return this.isEmbedded;
 	},
 });
 
 Template.adminFlex.events({
-	'click [data-action="close"]'() {
+	'click [data-action="close"]'(e, instance) {
+		if (instance.isEmbedded) {
+			menu.close();
+			return;
+		}
+
 		SideNav.closeFlex();
 	},
 	'keyup [name=settings-search]'(e, t) {
diff --git a/app/ui-master/client/main.js b/app/ui-master/client/main.js
index cbc36203f537..644607d467bd 100644
--- a/app/ui-master/client/main.js
+++ b/app/ui-master/client/main.js
@@ -133,9 +133,7 @@ Template.main.onCreated(function() {
 
 Template.main.helpers({
 	removeSidenav() {
-		const { center } = this;
-		const centerTemplate = (center || typeof center === 'function' ? center() : center);
-		return Layout.isEmbedded() && !/^admin/.test(centerTemplate);
+		return Layout.isEmbedded() && !/^\/admin/.test(FlowRouter.current().route.path);
 	},
 	siteName() {
 		return settings.get('Site_Name');
diff --git a/app/ui-utils/client/lib/SideNav.js b/app/ui-utils/client/lib/SideNav.js
index d24ad889fec6..5f4d720c9cd9 100644
--- a/app/ui-utils/client/lib/SideNav.js
+++ b/app/ui-utils/client/lib/SideNav.js
@@ -41,16 +41,10 @@ export const SideNav = new class {
 		}, 500);
 	}
 	closeFlex(callback = null) {
-		let subscription;
-		if (!roomTypes.getTypes().filter(function(i) {
-			return i.route;
-		}).map(function(i) {
-			return i.route.name;
-		}).includes(FlowRouter.current().route.name)) {
-			subscription = Subscriptions.findOne({
-				rid: Session.get('openedRoom'),
-			});
-			if (subscription != null) {
+		const routesNamesForRooms = roomTypes.getTypes().filter((i) => i.route).map((i) => i.route.name);
+		if (!routesNamesForRooms.includes(FlowRouter.current().route.name)) {
+			const subscription = Subscriptions.findOne({ rid: Session.get('openedRoom') });
+			if (subscription) {
 				roomTypes.openRouteLink(subscription.t, subscription, FlowRouter.current().queryParams);
 			} else {
 				FlowRouter.go('home');
@@ -59,7 +53,7 @@ export const SideNav = new class {
 		if (this.animating === true) {
 			return;
 		}
-		return this.toggleFlex(-1, callback);
+		this.toggleFlex(-1, callback);
 	}
 	flexStatus() {
 		return this.flexNav.opened;
diff --git a/client/routes/router.js b/client/routes/router.js
index f3b713b9ca2b..105cf8493f6b 100644
--- a/client/routes/router.js
+++ b/client/routes/router.js
@@ -7,10 +7,9 @@ import { FlowRouter } from 'meteor/kadira:flow-router';
 import { BlazeLayout } from 'meteor/kadira:blaze-layout';
 import { Session } from 'meteor/session';
 import { KonchatNotification } from '../../app/ui';
-import { Layout } from '../../app/ui-utils/client';
 
 Blaze.registerHelper('pathFor', function(path, kw) {
-	return FlowRouter.path(path, kw.hash, Layout.isEmbedded ? { layout: 'embedded' } : {});
+	return FlowRouter.path(path, kw.hash);
 });
 
 BlazeLayout.setRoot('body');