From d026bea07d72d759b7f5fd873c9c4b40d08ae573 Mon Sep 17 00:00:00 2001
From: dartcafe <github@dartcafe.de>
Date: Sat, 22 Feb 2020 12:51:13 +0100
Subject: [PATCH 1/6] changed ButtonStyle

---
 src/js/components/Base/ButtonDiv.vue               |  2 +-
 src/js/components/Navigation/Navigation.vue        |  2 +-
 .../components/SideBar/SideBarTabConfiguration.vue |  3 ++-
 .../components/SideBar/SideBarTabOptionsDate.vue   |  3 ++-
 src/js/components/VoteTable/VoteHeaderPublic.vue   | 14 ++++++++++++--
 src/js/views/Vote.vue                              |  6 +++++-
 6 files changed, 23 insertions(+), 7 deletions(-)

diff --git a/src/js/components/Base/ButtonDiv.vue b/src/js/components/Base/ButtonDiv.vue
index c4c40ac04..fe9ab05c6 100644
--- a/src/js/components/Base/ButtonDiv.vue
+++ b/src/js/components/Base/ButtonDiv.vue
@@ -54,7 +54,7 @@ export default {
 
 <style lang="scss" scoped>
 	.button {
-		display: inline-flex;
+		display: inline-block;
 
 		&.withIcon {
 			padding-left: 34px;
diff --git a/src/js/components/Navigation/Navigation.vue b/src/js/components/Navigation/Navigation.vue
index 9bd9c5afc..a79e000df 100644
--- a/src/js/components/Navigation/Navigation.vue
+++ b/src/js/components/Navigation/Navigation.vue
@@ -22,7 +22,7 @@
 
 <template lang="html">
 	<AppNavigation>
-		<AppNavigationNew :text="t('polls', 'Add new Poll')" @click="toggleCreateDlg" />
+		<AppNavigationNew class="icon-add" :text="t('polls', 'Add new Poll')" @click="toggleCreateDlg" />
 		<CreateDlg v-show="createDlg" ref="createDlg" @closeCreate="closeCreate()" />
 		<ul>
 			<AppNavigationItem :title="t('polls', 'All polls')" :allow-collapse="true"
diff --git a/src/js/components/SideBar/SideBarTabConfiguration.vue b/src/js/components/SideBar/SideBarTabConfiguration.vue
index 7740b6e12..a0bea02b4 100644
--- a/src/js/components/SideBar/SideBarTabConfiguration.vue
+++ b/src/js/components/SideBar/SideBarTabConfiguration.vue
@@ -75,7 +75,8 @@
 			<label for="public" class="title">{{ t('polls', 'Visible to other users') }} </label>
 		</div>
 
-		<ButtonDiv :primary="true" :title="poll.deleted ? t('polls', 'Restore poll') : t('polls', 'Delete poll')" @click="switchDeleted()" />
+		<ButtonDiv icon="icon-delete" :title="poll.deleted ? t('polls', 'Restore poll') : t('polls', 'Delete poll')"
+			@click="switchDeleted()" />
 	</div>
 </template>
 
diff --git a/src/js/components/SideBar/SideBarTabOptionsDate.vue b/src/js/components/SideBar/SideBarTabOptionsDate.vue
index b8c5594b8..92c995862 100644
--- a/src/js/components/SideBar/SideBarTabOptionsDate.vue
+++ b/src/js/components/SideBar/SideBarTabOptionsDate.vue
@@ -48,7 +48,8 @@
 				</div>
 			</div>
 			<div>
-				<ButtonDiv primary="true" :title="t('polls', 'Shift')" @click="shiftDates(move)" />
+				<ButtonDiv icon="icon-history" :title="t('polls', 'Shift')"
+					@click="shiftDates(move)" />
 			</div>
 		</div>
 
diff --git a/src/js/components/VoteTable/VoteHeaderPublic.vue b/src/js/components/VoteTable/VoteHeaderPublic.vue
index ea4d4a04c..ba35c0c8a 100644
--- a/src/js/components/VoteTable/VoteHeaderPublic.vue
+++ b/src/js/components/VoteTable/VoteHeaderPublic.vue
@@ -41,8 +41,10 @@
 					<span v-show="!checkingUserName && userName.length < 3">{{ t('polls', 'Username is not valid. Please enter at least 3 characters.') }}</span>
 					<span v-show="!checkingUserName && userName.length > 2 && !isValidName">{{ t('polls', 'This username is not valid, i.e. because it is already in use.') }}</span>
 				</div>
-				<ButtonDiv :primary="true" :disabled="!isValidName || checkingUserName" :title="t('polls', 'OK')"
-					@click="writeUserName" />
+				<div class="modal__buttons">
+					<ButtonDiv :primary="true" :disabled="!isValidName || checkingUserName" :title="t('polls', 'OK')"
+						@click="writeUserName" />
+				</div>
 			</div>
 		</Modal>
 	</div>
@@ -196,6 +198,14 @@ export default {
 			width: 100%;
 		}
 	}
+	.modal__buttons {
+		display: flex;
+		justify-content: end;
+		.button {
+			margin-left: 10px;
+			margin-right: 0;
+		}
+	}
 
 	.personal-link {
 		display: flex;
diff --git a/src/js/views/Vote.vue b/src/js/views/Vote.vue
index 3f2ac24cf..b73f929f8 100644
--- a/src/js/views/Vote.vue
+++ b/src/js/views/Vote.vue
@@ -24,7 +24,11 @@
 	<AppContent>
 		<div class="main-container">
 			<div class="header-actions">
-				<ButtonDiv :primary="true" :title="t('polls', 'Switch view')" @click="tableMode = !tableMode" />
+				<ButtonDiv
+					:primary="true"
+					:icon="tableMode ? 'icon-toggle-filelist' : 'icon-toggle-pictures'"
+					:title="t('polls', 'Switch view')"
+					@click="tableMode = !tableMode" />
 				<a href="#" class="icon icon-settings active"
 					:title="t('polls', 'Open Sidebar')" @click="toggleSideBar()" />
 			</div>

From 278674b9d12f9766a9ad5a397ecbaec0407295c5 Mon Sep 17 00:00:00 2001
From: dartcafe <github@dartcafe.de>
Date: Sat, 22 Feb 2020 16:24:29 +0100
Subject: [PATCH 2/6] change headerButtons to ActionButtons

---
 src/js/views/Vote.vue | 21 +++++++++++++--------
 1 file changed, 13 insertions(+), 8 deletions(-)

diff --git a/src/js/views/Vote.vue b/src/js/views/Vote.vue
index b73f929f8..c86ebca7c 100644
--- a/src/js/views/Vote.vue
+++ b/src/js/views/Vote.vue
@@ -24,13 +24,16 @@
 	<AppContent>
 		<div class="main-container">
 			<div class="header-actions">
-				<ButtonDiv
-					:primary="true"
-					:icon="tableMode ? 'icon-toggle-filelist' : 'icon-toggle-pictures'"
-					:title="t('polls', 'Switch view')"
-					@click="tableMode = !tableMode" />
-				<a href="#" class="icon icon-settings active"
-					:title="t('polls', 'Open Sidebar')" @click="toggleSideBar()" />
+				<Actions>
+					<ActionButton :icon="tableMode ? 'icon-toggle-filelist' : 'icon-toggle-pictures'" @click="tableMode = !tableMode">
+						{{ t('polls', 'Switch view') }}
+					</ActionButton>
+				</Actions>
+				<Actions>
+					<ActionButton icon="icon-settings" @click="toggleSideBar()">
+						{{ t('polls', 'Toggle Sidebar') }}
+					</ActionButton>
+				</Actions>
 			</div>
 			<PollTitle />
 			<PollInformation />
@@ -60,7 +63,7 @@
 </template>
 
 <script>
-import { AppContent } from '@nextcloud/vue'
+import { Actions, ActionButton, AppContent } from '@nextcloud/vue'
 import Subscription from '../components/Subscription/Subscription'
 import ParticipantsList from '../components/Base/ParticipantsList'
 import PollDescription from '../components/Base/PollDescription'
@@ -76,6 +79,8 @@ import { mapState } from 'vuex'
 export default {
 	name: 'Vote',
 	components: {
+		Actions,
+		ActionButton,
 		AppContent,
 		Subscription,
 		ParticipantsList,

From 67158fb13ccd8e2c52939fb615b8be8d9729eb69 Mon Sep 17 00:00:00 2001
From: dartcafe <github@dartcafe.de>
Date: Sat, 22 Feb 2020 16:29:45 +0100
Subject: [PATCH 3/6] adjust title padding on viewpoints < 1024px and fix #847

---
 src/js/views/Vote.vue | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/js/views/Vote.vue b/src/js/views/Vote.vue
index c86ebca7c..9244d2da6 100644
--- a/src/js/views/Vote.vue
+++ b/src/js/views/Vote.vue
@@ -35,7 +35,7 @@
 					</ActionButton>
 				</Actions>
 			</div>
-			<PollTitle />
+			<PollTitle class="poll-title" />
 			<PollInformation />
 			<VoteHeaderPublic v-if="!OC.currentUser" />
 			<PollDescription />
@@ -198,6 +198,7 @@ export default {
 
 .header-actions {
 	right: 0;
+	top: 0;
 	position: absolute;
 	display: flex;
 }
@@ -207,5 +208,10 @@ export default {
 	width: 44px;
 	height: 44px;
 }
-
+@media (max-width: (1024px)) {
+	.poll-title {
+		padding-left: 14px;
+		padding-right: 90px;
+	}
+}
 </style>

From 11c98c4ba508c230717e0a2a5405720f9ff101a1 Mon Sep 17 00:00:00 2001
From: dartcafe <github@dartcafe.de>
Date: Sat, 22 Feb 2020 16:35:23 +0100
Subject: [PATCH 4/6] added ButtonClass to AppNavigationNew

---
 src/js/components/Navigation/Navigation.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/js/components/Navigation/Navigation.vue b/src/js/components/Navigation/Navigation.vue
index a79e000df..6fba2853d 100644
--- a/src/js/components/Navigation/Navigation.vue
+++ b/src/js/components/Navigation/Navigation.vue
@@ -22,7 +22,7 @@
 
 <template lang="html">
 	<AppNavigation>
-		<AppNavigationNew class="icon-add" :text="t('polls', 'Add new Poll')" @click="toggleCreateDlg" />
+		<AppNavigationNew button-class="icon-add" :text="t('polls', 'Add new Poll')" @click="toggleCreateDlg" />
 		<CreateDlg v-show="createDlg" ref="createDlg" @closeCreate="closeCreate()" />
 		<ul>
 			<AppNavigationItem :title="t('polls', 'All polls')" :allow-collapse="true"

From a33ce510d9a84a4d3965807112f3286bdce10723 Mon Sep 17 00:00:00 2001
From: dartcafe <github@dartcafe.de>
Date: Sat, 22 Feb 2020 17:18:46 +0100
Subject: [PATCH 5/6] Changed share options to ActionButtons

---
 src/js/components/SideBar/SideBarTabShare.vue | 39 +++++++++++++------
 1 file changed, 27 insertions(+), 12 deletions(-)

diff --git a/src/js/components/SideBar/SideBarTabShare.vue b/src/js/components/SideBar/SideBarTabShare.vue
index a3928aa9d..a286aaafa 100644
--- a/src/js/components/SideBar/SideBarTabShare.vue
+++ b/src/js/components/SideBar/SideBarTabShare.vue
@@ -36,10 +36,14 @@
 					:display-name="shareDisplayName(share)"
 					:type="share.type"
 					:icon="true" />
-				<div class="options">
-					<a class="icon icon-clippy" @click="copyLink( { url: OC.generateUrl('apps/polls/s/') + share.token } )" />
-					<a class="icon icon-delete svg delete-poll" @click="removeShare(share)" />
-				</div>
+				<Actions>
+					<ActionButton icon="icon-clippy" @click="copyLink( { url: OC.generateUrl('apps/polls/s/') + share.token })">
+						{{ t('polls', 'Copy link to clipboard') }}
+					</ActionButton>
+					<ActionButton icon="icon-delete" @click="removeShare(share)">
+						{{ t('polls', 'Remove share') }}
+					</ActionButton>
+				</Actions>
 			</li>
 		</TransitionGroup>
 
@@ -76,24 +80,39 @@
 						{{ t('polls', 'Public link (' + share.token + ')') }}
 					</div>
 				</div>
-				<div class="options">
+				<Actions>
+					<ActionButton icon="icon-clippy" @click="copyLink( { url: OC.generateUrl('apps/polls/s/') + share.token })">
+						{{ t('polls', 'Copy link to clipboard') }}
+					</ActionButton>
+					<ActionButton icon="icon-delete" @click="removeShare(share)">
+						{{ t('polls', 'Remove share') }}
+					</ActionButton>
+				</Actions>
+				<!-- <div class="options">
 					<a class="icon icon-clippy" @click="copyLink( { url: OC.generateUrl('apps/polls/s/') + share.token } )" />
 					<a class="icon icon-delete" @click="removeShare(share)" />
-				</div>
+				</div> -->
 			</li>
 		</TransitionGroup>
-		<ButtonDiv icon="icon-add" :title="t('polls', 'Add a public link')" @click="addShare({type: 'public' })" />
+		<div class="user-row user" @click="addShare({type: 'public', user: '', emailAddress: ''})">
+			<div class="avatar icon-add" />
+			<div class="user-name">
+				{{ t('polls', 'Add a public link') }}
+			</div>
+		</div>
 	</div>
 </template>
 
 <script>
-import { Multiselect } from '@nextcloud/vue'
+import { Actions, ActionButton, Multiselect } from '@nextcloud/vue'
 import { mapState, mapGetters } from 'vuex'
 
 export default {
 	name: 'SideBarTabShare',
 
 	components: {
+		Actions,
+		ActionButton,
 		Multiselect
 	},
 
@@ -211,10 +230,6 @@ export default {
 </script>
 
 <style lang="scss">
-	.add * {
-		cursor: pointer;
-	}
-
 	.shared-list {
 		display: flex;
 		flex-wrap: wrap;

From c0f4b3b1bd5bfe711327e51a5ffad2d34ebb2919 Mon Sep 17 00:00:00 2001
From: dartcafe <github@dartcafe.de>
Date: Tue, 25 Feb 2020 07:02:57 +0100
Subject: [PATCH 6/6] added possibility to close modal on public poll

---
 src/js/components/VoteTable/VoteHeaderPublic.vue | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/js/components/VoteTable/VoteHeaderPublic.vue b/src/js/components/VoteTable/VoteHeaderPublic.vue
index ba35c0c8a..6a65f94d9 100644
--- a/src/js/components/VoteTable/VoteHeaderPublic.vue
+++ b/src/js/components/VoteTable/VoteHeaderPublic.vue
@@ -26,7 +26,7 @@
 			{{ t('polls', 'Your personal link to this poll: %n', 1, personalLink) }}
 			<a class="icon icon-clippy" @click="copyLink()" />
 		</div>
-		<Modal v-show="!isValidUser &!expired" class="modal"
+		<Modal v-show="!isValidUser &!expired & modal" class="modal"
 			:can-close="false">
 			<div class="modal__content">
 				<h2>{{ t('polls', 'Enter your name!') }}</h2>
@@ -42,6 +42,8 @@
 					<span v-show="!checkingUserName && userName.length > 2 && !isValidName">{{ t('polls', 'This username is not valid, i.e. because it is already in use.') }}</span>
 				</div>
 				<div class="modal__buttons">
+					<ButtonDiv :title="t('polls', 'Cancel')"
+						@click="closeModal" />
 					<ButtonDiv :primary="true" :disabled="!isValidName || checkingUserName" :title="t('polls', 'OK')"
 						@click="writeUserName" />
 				</div>
@@ -71,7 +73,7 @@ export default {
 			redirecting: false,
 			isValidName: false,
 			newName: '',
-			modal: false
+			modal: true
 		}
 	},