From 9e7e027fe9effac40561da7e5ad10928111d9b93 Mon Sep 17 00:00:00 2001
From: Stefan Zermatten <stefanzermatten@gmail.com>
Date: Wed, 29 Apr 2015 08:38:47 +0200
Subject: [PATCH] Share dialog style improvement, can delete shares

---
 .../characterSettings/shareDialog.html        | 25 +++++++++++++------
 .../characterSettings/shareDialog.js          |  5 ++++
 2 files changed, 23 insertions(+), 7 deletions(-)

diff --git a/rpg-docs/client/views/character/characterSettings/shareDialog.html b/rpg-docs/client/views/character/characterSettings/shareDialog.html
index 84b6e497..9ce9b6ff 100644
--- a/rpg-docs/client/views/character/characterSettings/shareDialog.html
+++ b/rpg-docs/client/views/character/characterSettings/shareDialog.html
@@ -1,12 +1,15 @@
 <template name="shareDialog">
-	<div>
+	<div style="width: 360px;">
 		<div>
 			{{#if readers.count}}
 				<div style="font-weight: 500;">
 					Can View
 				</div>
 				{{#each readers}}
-					{{username}}<br>
+					<div layout horizontal center>
+						<div flex>{{username}}</div>
+						<paper-icon-button class="deleteShare" icon="delete"></paper-icon-button>
+					</div>
 				{{/each}}
 			{{/if}}
 			{{#if writers.count}}
@@ -14,18 +17,26 @@
 					Can Edit
 				</div>
 				{{#each writers}}
-					{{username}}<br>
+					<div layout horizontal center>
+						<div flex>{{username}}</div>
+						<paper-icon-button class="deleteShare" icon="delete"></paper-icon-button>
+					</div>
 				{{/each}}
 			{{/if}}
 		</div>
-		<paper-input id="userNameOrEmailInput" label="Username or email" floatinglabel></paper-input><br>
-		{{#if userFindError}}<p style="color: red;">{{userFindError}}</p>{{/if}}
+		<div layout horizontal center>
+			<paper-input flex id="userNameOrEmailInput" label="Username or email" floatinglabel></paper-input>
+			<paper-button id="shareButton"
+						  class="red-button"
+						  style="width: 80px; height: 37px; margin-top: 16px;"
+						  raised
+						  disabled={{shareButtonDisabled}}>Share</paper-button>
+		</div>
+		<p style="color: red;">{{userFindError}}</p>
 		<paper-radio-group id="accessLevelMenu" selected="read">
 			<paper-radio-button name="read" label="View Only"></paper-radio-button>
 			<paper-radio-button name="write" label="Can Edit"></paper-radio-button>
 		</paper-radio-group>
-		<br>
-		<paper-button id="shareButton" class="red-button" raised disabled={{shareButtonDisabled}}>Share</paper-button>
 	</div>
 	<paper-button id="doneButton" affirmative> Done </paper-button>
 </template>
diff --git a/rpg-docs/client/views/character/characterSettings/shareDialog.js b/rpg-docs/client/views/character/characterSettings/shareDialog.js
index b03ed143..31efe05c 100644
--- a/rpg-docs/client/views/character/characterSettings/shareDialog.js
+++ b/rpg-docs/client/views/character/characterSettings/shareDialog.js
@@ -53,4 +53,9 @@ Template.shareDialog.events({
 			});
 		}
 	},
+	"tap .deleteShare": function(event, instance) {
+		Characters.update(instance.data._id, {
+			$pull: {writers: this._id, readers: this._id}
+		});
+	},
 });