From f86c9cf853b3920009f9a8f942d48cb17d69e2e9 Mon Sep 17 00:00:00 2001
From: James Allen <james@sharelatex.com>
Date: Wed, 2 Jul 2014 10:59:18 +0100
Subject: [PATCH] Move all editor manager binding logic into the managers

---
 services/web/app/views/project/editor.jade     |  2 +-
 .../app/views/project/editor/left-menu.jade    | 14 ++++++++++++++
 .../app/ide/editor/directives/aceEditor.coffee |  7 ++-----
 .../editor/highlights/HighlightsManager.coffee |  3 +++
 .../stylesheets/app/editor/left-menu.less      | 18 ++++++++++++++++++
 .../public/stylesheets/app/editor/toolbar.less |  3 ++-
 6 files changed, 40 insertions(+), 7 deletions(-)

diff --git a/services/web/app/views/project/editor.jade b/services/web/app/views/project/editor.jade
index 1fba8d539e..d0224b6d1f 100644
--- a/services/web/app/views/project/editor.jade
+++ b/services/web/app/views/project/editor.jade
@@ -8,7 +8,7 @@ block vars
 block scripts
 	//- Only use the native bootstrap on the editor page,
 	//- since we use the Angular-based bootstrap elsewhere.
-	script(src=jsPath+'libs/bootstrap-3.1.1.js')
+	//- script(src=jsPath+'libs/bootstrap-3.1.1.js')
 	script(src=jsPath+'libs/jquery-layout.js')
 	script(src=jsPath+'libs/jquery.storage.js')
 
diff --git a/services/web/app/views/project/editor/left-menu.jade b/services/web/app/views/project/editor/left-menu.jade
index 16732611b0..717d6d9287 100644
--- a/services/web/app/views/project/editor/left-menu.jade
+++ b/services/web/app/views/project/editor/left-menu.jade
@@ -2,6 +2,20 @@ aside#left-menu.full-size(
 	ng-class="{ 'shown': ui.leftMenuShown }"
 	ng-cloak
 )
+	h4 Download
+
+	ul.unformatted-list.nav.nav-downloads
+		li
+			a(ng-href="/project/{{project_id}}/download/zip")
+				i.fa.fa-file-archive-o.fa-2x
+				br
+				| Source
+		li
+			a(ng-href="/project/{{project_id}}/output/output.pdf")
+				i.fa.fa-file-pdf-o.fa-2x
+				br
+				| PDF
+
 	h4 Settings
 	form.settings(ng-controller="SettingsController")
 		.containter-fluid
diff --git a/services/web/public/coffee/app/ide/editor/directives/aceEditor.coffee b/services/web/public/coffee/app/ide/editor/directives/aceEditor.coffee
index 594dd632f8..423f9a1307 100644
--- a/services/web/public/coffee/app/ide/editor/directives/aceEditor.coffee
+++ b/services/web/public/coffee/app/ide/editor/directives/aceEditor.coffee
@@ -10,7 +10,7 @@ define [
 	"ace/keyboard/emacs"
 	"ace/mode/latex"
 	"ace/edit_session"
-], (App, Ace, UndoManager, AutoCompleteManager, SpellCheckManager, AnnotationsManager, CursorPositionManager) ->
+], (App, Ace, UndoManager, AutoCompleteManager, SpellCheckManager, HighlightsManager, CursorPositionManager) ->
 	LatexMode = require("ace/mode/latex").Mode
 	EditSession = require('ace/edit_session').EditSession
 
@@ -49,7 +49,7 @@ define [
 				autoCompleteManager   = new AutoCompleteManager(scope, editor, element)
 				spellCheckManager     = new SpellCheckManager(scope, editor, element)
 				undoManager           = new UndoManager(scope, editor, element)
-				annotationsManager    = new AnnotationsManager(scope, editor, element)
+				highlightsManager     = new HighlightsManager(scope, editor, element)
 				cursorPositionManager = new CursorPositionManager(scope, editor, element)
 
 				# Prevert Ctrl|Cmd-S from triggering save dialog
@@ -119,9 +119,6 @@ define [
 					resetSession()
 					session = editor.getSession()
 
-					autoCompleteManager.bindToSession(session)
-					annotationsManager.redrawAnnotations()
-
 					doc = session.getDocument()
 					doc.on "change", () ->
 						scope.$apply () ->
diff --git a/services/web/public/coffee/app/ide/editor/highlights/HighlightsManager.coffee b/services/web/public/coffee/app/ide/editor/highlights/HighlightsManager.coffee
index b1afe3fb57..18d205c448 100644
--- a/services/web/public/coffee/app/ide/editor/highlights/HighlightsManager.coffee
+++ b/services/web/public/coffee/app/ide/editor/highlights/HighlightsManager.coffee
@@ -29,6 +29,9 @@ define [
 				e.position = position
 				@showAnnotationLabels(position)
 
+			@editor.on "changeSession", () =>
+				@redrawAnnotations()
+
 		redrawAnnotations: () ->
 			@_clearMarkers()
 			@_clearLabels()
diff --git a/services/web/public/stylesheets/app/editor/left-menu.less b/services/web/public/stylesheets/app/editor/left-menu.less
index 140d70e9fb..7d90ebedba 100644
--- a/services/web/public/stylesheets/app/editor/left-menu.less
+++ b/services/web/public/stylesheets/app/editor/left-menu.less
@@ -31,6 +31,24 @@
 		margin-top: 0;
 	}
 
+	ul.nav-downloads {
+		li {
+			display: inline-block;
+			text-align: center;
+			width: 100px;
+			a {
+				color: @gray-dark;
+				&:hover, &:active {
+					background-color: @link-color;
+					color: white;
+				}
+				i {
+					margin-bottom: (@line-height-computed / 4);
+				}
+			}
+		}
+	}
+
 	form.settings {
 		label {
 			font-weight: normal;
diff --git a/services/web/public/stylesheets/app/editor/toolbar.less b/services/web/public/stylesheets/app/editor/toolbar.less
index 81b7188611..ba3f667d9a 100644
--- a/services/web/public/stylesheets/app/editor/toolbar.less
+++ b/services/web/public/stylesheets/app/editor/toolbar.less
@@ -25,7 +25,8 @@
 		border-radius: 0;
 		border-right: 1px solid @toolbar-border-color;
 		color: @link-color;
-		padding: 6px 12px 8px;
+		padding: 3px 10px 5px;
+		font-size: 20px;
 		&:hover {
 			text-shadow: 0 1px 0 rgba(0, 0, 0, 0.15);
 			background-color: darken(white, 10%);