Allow subscribers to control the presentation, too.

static/js/directives/pdfcanvas.js

@@ -93,6 +93,10 @@ define(['require', 'underscore', 'jquery', 'pdf'], function(require, _, $, pdf)
 			};
 
 			PDFCanvas.prototype._showPage = function(page) {
+				if (page === this.scope.currentPageNumber) {
+					return;
+				}
+
 				console.log("Showing page", page, "/", this.scope.maxPageNumber);
 				if (this.currentPage) {
 					this.currentPage.destroy();

static/js/directives/presentation.js

@@ -29,7 +29,7 @@ define(['jquery', 'underscore', 'text!partials/presentation.html'], function($,
 
 			$scope.layout.presentation = false;
 			$scope.isPresenter = false;
-			$scope.hideControlsBar = false;
+			$scope.hideControlsBar = true;
 			$scope.pendingPageRequest = null;
 			$scope.presentationLoaded = false;
 
@@ -180,17 +180,12 @@ define(['jquery', 'underscore', 'text!partials/presentation.html'], function($,
 			};
 
 			$scope.$on("pdfPageLoading", function(event, page) {
-				if (!$scope.isPresenter) {
-					return;
-				}
-
-				_.each(peers, function(ignore, peerId) {
-					var peercall = mediaStream.webrtc.findTargetCall(peerId);
+				mediaStream.webrtc.callForEachCall(function(peercall) {
 					mediaStream.api.apply("sendPresentation", {
 						send: function(type, data) {
 							return peercall.peerconnection.send(data);
 						}
-					})(peerId, currentToken, {
+					})(peercall.id, currentToken, {
 						Type: "Page",
 						Page: page
 					});
@@ -261,6 +256,7 @@ define(['jquery', 'underscore', 'text!partials/presentation.html'], function($,
 						});
 						uploadPresentation(info);
 						$scope.isPresenter = true;
+						$scope.hideControlsBar = false;
 						$scope.$emit("openPdf", f);
 					}, this));
 				}, this));
@@ -278,6 +274,7 @@ define(['jquery', 'underscore', 'text!partials/presentation.html'], function($,
 				finishDownloadPresentation();
 				$scope.layout.presentation = false;
 				$scope.isPresenter = false;
+				$scope.hideControlsBar = true;
 				$scope.$emit("mainview", "presentation", false);
 			};
 

static/partials/presentation.html

@@ -1,12 +1,12 @@
 <div class="presentation">
 	<div class="presentationpane nicescroll">
-        <pdfcanvas/>
-    </div>
-	<div class="overlaybar form-horizontal" ng-class="{notvisible: hideControlsBar}" ng-show="isPresenter">
+		<pdfcanvas/>
+	</div>
+	<div class="overlaybar form-horizontal" ng-class="{notvisible: hideControlsBar}">
 		<a class="overlaybar-button" ng-model="hideControlsBar" btn-checkbox btn-checkbox btn-checkbox-true="0" btn-checkbox-false="1" title="{{_('Presentation controls')}}"><i class="fa fa-cogs"></i></a>
 		<form class="overlaybar-content form-group">
-            <a class="btn" ng-click="prevPage()" ng-show="currentPageNumber &gt; 1" title="{{_('Prev')}}"><i class="fa fa-backward"></i></a>
-            <a class="btn" ng-click="nextPage()" ng-show="currentPageNumber &lt; maxPageNumber" title="{{_('Next')}}"><i class="fa fa-forward"></i></a>
+			<a class="btn" ng-click="prevPage()" ng-show="currentPageNumber &gt; 1" title="{{_('Prev')}}"><i class="fa fa-backward"></i></a>
+			<a class="btn" ng-click="nextPage()" ng-show="currentPageNumber &lt; maxPageNumber" title="{{_('Next')}}"><i class="fa fa-forward"></i></a>
 		</form>
 	</div>
 </div>