From 9ed03fb7eb16314a2b7a3344344b09962567cd65 Mon Sep 17 00:00:00 2001
From: Tasso Evangelista <tasso.evangelista@rocket.chat>
Date: Tue, 23 Apr 2019 19:51:46 -0300
Subject: [PATCH 1/3] Only show sidenav in embedded mode if rendering admin
 templates

---
 app/theme/client/imports/general/base_old.css |  6 ---
 app/ui-master/client/main.html                |  2 +-
 app/ui-master/client/main.js                  |  9 ++--
 app/ui-sidenav/client/sideNav.html            | 52 +++++++++----------
 app/ui-utils/client/lib/openRoom.js           |  5 +-
 app/ui/client/components/header/header.html   |  3 +-
 6 files changed, 34 insertions(+), 43 deletions(-)

diff --git a/app/theme/client/imports/general/base_old.css b/app/theme/client/imports/general/base_old.css
index c5e9f3cb4064..1788ca21c93d 100644
--- a/app/theme/client/imports/general/base_old.css
+++ b/app/theme/client/imports/general/base_old.css
@@ -1397,12 +1397,6 @@ rc-old select,
 /* MAIN CONTENT + MAIN PAGES */
 
 .rc-old.main-content {
-	&.main-modal {
-		left: 0;
-
-		margin-right: 0;
-	}
-
 	& .container-fluid {
 		padding-top: 0;
 	}
diff --git a/app/ui-master/client/main.html b/app/ui-master/client/main.html
index 4436cf77efb8..fa1554c3c4b9 100644
--- a/app/ui-master/client/main.html
+++ b/app/ui-master/client/main.html
@@ -37,7 +37,7 @@
 								{{#unless removeSidenav}}
 									{{> sideNav }}
 								{{/unless}}
-								<div class="{{#unless $eq old false}} rc-old {{/unless}} main-content content-background-color {{readReceiptsEnabled}} {{#if modal}}main-modal{{/if}}">
+								<div class="{{#unless $eq old false}}rc-old{{/unless}} main-content content-background-color {{readReceiptsEnabled}}">
 									{{> Template.dynamic template=center}}
 								</div>
 							</div>
diff --git a/app/ui-master/client/main.js b/app/ui-master/client/main.js
index c5fad0cba14d..cbc36203f537 100644
--- a/app/ui-master/client/main.js
+++ b/app/ui-master/client/main.js
@@ -1,3 +1,5 @@
+import Clipboard from 'clipboard';
+import s from 'underscore.string';
 import { Meteor } from 'meteor/meteor';
 import { Match } from 'meteor/check';
 import { Tracker } from 'meteor/tracker';
@@ -13,8 +15,6 @@ import { CachedChatSubscription, Roles, ChatSubscription } from '../../models';
 import { CachedCollectionManager } from '../../ui-cached-collection';
 import { hasRole } from '../../authorization';
 import { tooltip } from '../../tooltip';
-import Clipboard from 'clipboard';
-import s from 'underscore.string';
 
 settings.collection.find({ _id:/theme-color-rc/i }, { fields:{ value: 1 } }).observe({ changed: () => { DynamicCss.run(true, settings); } });
 
@@ -133,8 +133,9 @@ Template.main.onCreated(function() {
 
 Template.main.helpers({
 	removeSidenav() {
-		const { modal } = this;
-		return (modal || typeof modal === 'function' ? modal() : modal); // || RocketChat.Layout.isEmbedded();
+		const { center } = this;
+		const centerTemplate = (center || typeof center === 'function' ? center() : center);
+		return Layout.isEmbedded() && !/^admin/.test(centerTemplate);
 	},
 	siteName() {
 		return settings.get('Site_Name');
diff --git a/app/ui-sidenav/client/sideNav.html b/app/ui-sidenav/client/sideNav.html
index d5aa78944ffa..8eed7aa179d8 100644
--- a/app/ui-sidenav/client/sideNav.html
+++ b/app/ui-sidenav/client/sideNav.html
@@ -1,30 +1,30 @@
 <template name="sideNav">
-		<aside class="sidebar sidebar--{{sidebarViewMode}} {{#if sidebarHideAvatar}}sidebar--hide-avatar{{/if}}" role="navigation">
-			{{> sidebarHeader }}
-			{{#if loggedInUser}}
-				<div class="unread-rooms background-primary-action-color color-primary-action-contrast top-unread-rooms hidden">
-					{{_ "More_unreads"}} <i class="icon-up-big"></i>
+	<aside class="sidebar sidebar--{{sidebarViewMode}} {{#if sidebarHideAvatar}}sidebar--hide-avatar{{/if}}" role="navigation">
+		{{> sidebarHeader }}
+		{{#if loggedInUser}}
+			<div class="unread-rooms background-primary-action-color color-primary-action-contrast top-unread-rooms hidden">
+				{{_ "More_unreads"}} <i class="icon-up-big"></i>
+			</div>
+			<div class="rooms-list" aria-label="{{_ "Channels"}}" role="region">
+				{{#each roomType}}
+					{{> Template.dynamic template=template data=data }}
+				{{/each}}
+			</div>
+			<div class="unread-rooms background-primary-action-color color-primary-action-contrast bottom-unread-rooms hidden">
+				{{_ "More_unreads"}} <i class="icon-down-big"></i>
+			</div>
+			<div class="flex-nav animated-hidden">
+				{{> Template.dynamic template=flexTemplate data=flexData }}
+			</div>
+		{{else}}
+			<div class="rooms-list" aria-label="{{_ "Channels"}}" role="region">
+				<div class="wrapper">
+					{{> roomList anonymous=true label="Channels" }}
 				</div>
-				<div class="rooms-list" aria-label="{{_ "Channels"}}" role="region">
-					{{#each roomType}}
-						{{> Template.dynamic template=template data=data }}
-					{{/each}}
-				</div>
-				<div class="unread-rooms background-primary-action-color color-primary-action-contrast bottom-unread-rooms hidden">
-					{{_ "More_unreads"}} <i class="icon-down-big"></i>
-				</div>
-				<div class="flex-nav animated-hidden">
-					{{> Template.dynamic template=flexTemplate data=flexData }}
-				</div>
-			{{else}}
-				<div class="rooms-list" aria-label="{{_ "Channels"}}" role="region">
-					<div class="wrapper">
-						{{> roomList anonymous=true label="Channels" }}
-					</div>
-				</div>
-			{{/if}}
+			</div>
+		{{/if}}
 
-			<footer class="sidebar__footer">{{{footer}}}</footer>
-		</aside>
-		<div class="sidebar-wrap"></div>
+		<footer class="sidebar__footer">{{{footer}}}</footer>
+	</aside>
+	<div class="sidebar-wrap"></div>
 </template>
diff --git a/app/ui-utils/client/lib/openRoom.js b/app/ui-utils/client/lib/openRoom.js
index 4a02ff039049..c3a0e7a42795 100644
--- a/app/ui-utils/client/lib/openRoom.js
+++ b/app/ui-utils/client/lib/openRoom.js
@@ -5,7 +5,7 @@ import { Template } from 'meteor/templating';
 import { FlowRouter } from 'meteor/kadira:flow-router';
 import { BlazeLayout } from 'meteor/kadira:blaze-layout';
 import { Session } from 'meteor/session';
-import { Layout, RoomManager, fireGlobalEvent, readMessage, RoomHistoryManager } from '..';
+import { RoomManager, fireGlobalEvent, readMessage, RoomHistoryManager } from '..';
 import { ChatSubscription, Rooms } from '../../../models';
 import { settings } from '../../../settings';
 import { callbacks } from '../../../callbacks';
@@ -41,9 +41,6 @@ function replaceCenterDomBy(dom) {
 	return mainNode;
 }
 
-const { render } = BlazeLayout;
-BlazeLayout.render = (layout, regions = {}) => render(layout, Object.assign(regions, { modal: Layout.isEmbedded() }));
-
 export const openRoom = function(type, name) {
 	Session.set('openedRoom', null);
 
diff --git a/app/ui/client/components/header/header.html b/app/ui/client/components/header/header.html
index 086722c69aeb..14fb89cd3939 100644
--- a/app/ui/client/components/header/header.html
+++ b/app/ui/client/components/header/header.html
@@ -1,7 +1,6 @@
 <template name="header">
-<header class="rc-header">
+	<header class="rc-header">
 		<div class="rc-header__wrap">
-
 			<div class="rc-header__block rc-header--burger">
 				{{> burger}}
 			</div>

From 93a780344280446107031fae6be7c24ef4578057 Mon Sep 17 00:00:00 2001
From: Tasso Evangelista <tasso.evangelista@rocket.chat>
Date: Tue, 23 Apr 2019 20:34:55 -0300
Subject: [PATCH 2/3] Add layout parameter to pathFor helper

---
 client/routes/router.js | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/client/routes/router.js b/client/routes/router.js
index 9c17096e8e00..f3b713b9ca2b 100644
--- a/client/routes/router.js
+++ b/client/routes/router.js
@@ -1,3 +1,4 @@
+import s from 'underscore.string';
 import { Meteor } from 'meteor/meteor';
 import { Accounts } from 'meteor/accounts-base';
 import { Tracker } from 'meteor/tracker';
@@ -6,10 +7,10 @@ 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 s from 'underscore.string';
+import { Layout } from '../../app/ui-utils/client';
 
 Blaze.registerHelper('pathFor', function(path, kw) {
-	return FlowRouter.path(path, kw.hash);
+	return FlowRouter.path(path, kw.hash, Layout.isEmbedded ? { layout: 'embedded' } : {});
 });
 
 BlazeLayout.setRoot('body');

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 3/3] 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');