Merge pull request #554 from nextcloud/bugfix/550/make-14-compatible

Compatibility for Nextcloud 14

css/comp-13.scss

@@ -0,0 +1,38 @@
+#content-wrapper #content {
+	height: 100%;
+}
+#app-content {
+	flex-grow: 1;
+	height: 100%;
+
+	&.details-visible {
+		margin-right: 500px;
+	}
+}
+
+#app-sidebar {
+	right: -500px;
+	max-width: 100%;
+	width: 500px;
+	display:flex;
+	flex-direction: column;
+
+	&.details-visible {
+		right: 0;
+	}
+}
+
+#content[class*='app-'] * {
+	box-sizing: border-box;
+}
+
+body:not(.snapjs-left) {
+	.app-navigation-hide {
+		#app-content {
+			margin-left: 0 !important;
+		}
+		#app-navigation {
+			display: none;
+		}
+	}
+}

css/style.scss

@@ -70,6 +70,32 @@ input.input-inline {
 	cursor: text;
 }
 
+/**
+ * Generic app layout
+ */
+#content-wrapper {
+	overflow: hidden;
+	position: fixed;
+	width: 100%;
+	height: 100%;
+
+	#content {
+		height: calc(100% - 50px);
+		min-height: initial;
+	}
+
+	.app.app-deck {
+		width: 100%;
+		height: 100%;
+		display: flex;
+	}
+
+	#app-content {
+		display: flex;
+		flex-direction: column;
+	}
+}
+
 /**
  * Navigation sidebar
  */
@@ -238,7 +264,7 @@ input.input-inline {
 	}
 }
 
-#app-navigation-toggle {
+#app-navigation-toggle-custom {
 	width: 44px;
 	height: 44px;
 	cursor: pointer;
@@ -617,17 +643,6 @@ input.input-inline {
 /**
  * App sidebar
  */
-#app-sidebar {
- 	right: -500px;
- 	max-width: 100%;
- 	width: 500px;
- 	display:flex;
- 	flex-direction: column;
-
-	&.details-visible {
-	 	right: 0;
-	}
-}
 
 #sidebar-header {
 	h3 {
@@ -922,16 +937,6 @@ input.input-inline {
 	}
 }
 
-#app-content {
-	overflow: hidden;
-	display: flex;
-	flex-direction: column;
-
-	&.details-visible {
-		margin-right: 500px;
-	}
-}
-
 .labels {
 	display: block;
 	overflow: hidden;
@@ -1434,3 +1439,22 @@ input.input-inline {
 .ui-select-dropdown.select2-drop-active {
 	opacity: 1 !important;
 }
+
+/**
+ * Custom app sidebar handling
+ */
+body:not(.snapjs-left) {
+	.app-navigation-hide {
+		#app-content {
+			margin-left: 0 !important; /* overwrite margin since we want the translateX to handle it*/
+		}
+		#app-navigation {
+			transform: translateX(-300px);
+		}
+	}
+}
+@media only screen and (max-width: 768px) {
+	#app-navigation-toggle-custom {
+		display: none;
+	}
+}

js/app/Config.js

@@ -71,8 +71,9 @@ app.config(function ($provide, $interpolateProvider, $httpProvider, $urlRouterPr
 				tab: {value: 0, dynamic: true},
 			},
 			views: {
-				'sidebarView': {
+				'sidebarView@': {
-					templateUrl: '/board.sidebarView.html'
+					templateUrl: '/board.sidebarView.html',
+					controller: 'BoardController'
 				}
 			}
 		})
@@ -82,7 +83,7 @@ app.config(function ($provide, $interpolateProvider, $httpProvider, $urlRouterPr
 				tab: {value: 0, dynamic: true},
 			},
 			views: {
-				'sidebarView': {
+				'sidebarView@': {
 					templateUrl: '/card.sidebarView.html',
 					controller: 'CardController'
 				}

js/app/Run.js

@@ -56,26 +56,6 @@ app.run(function ($document, $rootScope, $transitions, BoardService) {
 		OC.filePath('deck', 'img', 'app-512.png')
 	);
 
-	$('#app-navigation-toggle').off('click');
-	// App sidebar on mobile
-	var snapper = new Snap({
-		element: document.getElementById('app-content'),
-		disable: 'right',
-		maxPosition: 250,
-		touchToDrag: false
-	});
-
-	$('#app-navigation-toggle').click(function () {
-		if ($(window).width() > 768) {
-			$('#app-navigation').toggle('hidden');
-		} else {
-			if (snapper.state().state === 'left') {
-				snapper.close();
-			} else {
-				snapper.open('left');
-			}
-		}
-	});
 	// Select all elements with data-toggle="tooltips" in the document
 	$('body').tooltip({
 		selector: '[data-toggle="tooltip"]'

js/controller/AppController.js

@@ -30,4 +30,13 @@ app.controller('AppController', function ($scope, $location, $http, $log, $rootS
 	$scope.sidebar = $rootScope.sidebar;
 	$scope.user = oc_current_user;
 	$rootScope.config = JSON.parse($attrs.config);
+
+	$scope.appNavigationHide = false;
+
+	$scope.toggleSidebar = function() {
+		if ($(window).width() > 768) {
+			$scope.appNavigationHide = !$scope.appNavigationHide;
+			console.log($scope.appNavigationHide);
+		}
+	};
 });

templates/main.php

@@ -28,32 +28,30 @@ Util::addScript('deck', 'build/vendor');
 
 Util::addStyle('deck', 'style');
 Util::addScript('deck', 'build/deck');
+
+if (\OC_Util::getVersion()[0] < 14) {
+	Util::addStyle('deck', 'comp-13');
+}
 ?>
 
-<div id="app" class="app-deck" data-ng-app="Deck" ng-controller="AppController" ng-cloak config="<?php p(json_encode($_)); ?>">
+<div class="app app-deck" data-ng-app="Deck" ng-controller="AppController" ng-cloak config="<?php p(json_encode($_)); ?>" ng-class="{'app-navigation-hide': appNavigationHide}">
 
 	<div id="app-navigation" data-ng-controller="ListController" ng-init="initSidebar()">
 		<?php print_unescaped($this->inc('part.navigation')); ?>
 		<?php /* print_unescaped($this->inc('part.settings')); */ ?>
 	</div>
-	<div id="app-content" ng-class="{ 'details-visible': sidebar.show }">
+	<div id="app-content" ng-class="{ 'details-visible': sidebar.show }"><div id="app-navigation-toggle-custom" class="icon-menu" ng-click="toggleSidebar()"></div><div ui-view></div></div>
-		<div ui-view></div>
+	<div id="app-sidebar" ng-class="{ 'details-visible': sidebar.show }" ng-if="sidebar.show" class="details-view scroll-container" ui-view="sidebarView"></div>
-	</div>
 	<route-loading-indicator></route-loading-indicator>
 
-
-
 	<script type="text/ng-template" id="/boardlist.mainView.html">
 		<?php print_unescaped($this->inc('part.boardlist')); ?>
 	</script>
 	<script type="text/ng-template" id="/board.sidebarView.html">
 		<?php print_unescaped($this->inc('part.board.sidebarView')); ?>
 	</script>
-	<script type="text/ng-template" id="/board.mainView.html">
-		<?php print_unescaped($this->inc('part.board.mainView')); ?>
-	</script>
 	<script type="text/ng-template" id="/board.html">
-		<?php print_unescaped($this->inc('part.board')); ?>
+		<?php print_unescaped($this->inc('part.board.mainView')); ?>
 	</script>
 	<script type="text/ng-template" id="/card.sidebarView.html">
 		<?php print_unescaped($this->inc('part.card')); ?>

templates/part.board.mainView.php

@@ -37,7 +37,6 @@
 </div>
 
 <div id="board" class="scroll-container" ng-click="sidebar.show=false" ui-sref="board" ng-class="{'card-selected': params.cardId}">
- {{ cardOpen }}
 	<search on-search="search" class="ng-hide"></search>
 
 	<div id="innerBoard" data-ng-model="stacks" data-as-sortable="sortOptionsStack">

templates/part.board.php

@@ -1,4 +0,0 @@
-<?php print_unescaped($this->inc('part.board.mainView')); ?>
-<div id="app-sidebar" class="details-view scroll-container"
-	 ng-class="{ 'details-visible': sidebar.show }" ui-view="sidebarView">
-</div>

templates/part.card.php

@@ -1,5 +1,5 @@
-<div nv-file-drop="" uploader="uploader" class="drop-zone" options="{cardId: cardservice.getCurrent().id}">
+<div nv-file-drop="" uploader="fileservice.uploader" class="drop-zone" options="{cardId: cardservice.getCurrent().id}">
-	<div class="drop-indicator" nv-file-over uploader="uploader">
+	<div class="drop-indicator" nv-file-over uploader="fileservice.uploader">
 		<p><?php p($l->t('Drop your files here to upload it to the card')); ?></p>
 	</div>
 	<div id="board-status" ng-if="statusservice.active">
@@ -97,8 +97,8 @@
 			<span class="save-indicator saved"><?php p($l->t('Saved')); ?></span>
 			<span class="save-indicator unsaved"><?php p($l->t('Unsaved changes')); ?></span>
 			<a ng-if="params.tab === 0" href="https://github.com/nextcloud/deck/wiki/Markdown-Help" target="_blank" class="icon icon-help" data-toggle="tooltip" data-placement="left" title="<?php p($l->t('Formatting help')); ?>"><span class="hidden-visually"><?php p($l->t('Formatting help')); ?></span></a>
-			<label ng-if="params.tab === 1" for="attachment-upload" class="button icon-upload" ng-class="{'icon-loading-small': uploader.isUploading}" data-toggle="tooltip" data-placement="left" title="<?php p($l->t('Upload attachment')); ?>"></label>
+			<label ng-if="params.tab === 1" for="attachment-upload" class="button icon-upload" ng-class="{'icon-loading-small': fileservice.uploader.isUploading}" data-toggle="tooltip" data-placement="left" title="<?php p($l->t('Upload attachment')); ?>"></label>
-			<input id="attachment-upload" type="file" nv-file-select="" uploader="uploader" class="hidden" options="{cardId: cardservice.getCurrent().id}"/>
+			<input id="attachment-upload" type="file" nv-file-select="" uploader="fileservice.uploader" class="hidden" options="{cardId: cardservice.getCurrent().id}"/>
 			<input ng-if="status.cardEditDescription" type="button" ng-mousedown="status.continueEdit = true; status.selectAttachment = true;" class="icon-files-dark" data-toggle="tooltip" data-placement="left" title="<?php p($l->t('Insert attachment')); ?>"/>
 
 		</div>