From c621258be3b4362cbc3bf5fc4277965e1729c4f4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?John=20Molakvo=C3=A6=20=28skjnldsv=29?=
 <skjnldsv@protonmail.com>
Date: Fri, 31 Aug 2018 07:23:58 +0200
Subject: Fixed grid
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
---
 src/components/ContactDetails.vue            | 6 +++---
 src/components/Properties/PropertyGroups.vue | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)

(limited to 'src')

diff --git a/src/components/ContactDetails.vue b/src/components/ContactDetails.vue
index cae7cd9c..c2bf5d54 100644
--- a/src/components/ContactDetails.vue
+++ b/src/components/ContactDetails.vue
@@ -86,9 +86,9 @@
 					:sorted-properties="sortedProperties" :property="property" :contact="contact"
 					@updatedcontact="updateContact" />
 
-				<!-- addressbook change select -->
-				<property-select :prop-model="addressbookModel" :value.sync="addressbook"
-					:options="addressbooksOptions" class="property--addressbooks" />
+				<!-- addressbook change select - no last property because class is not applied here-->
+				<property-select :prop-model="addressbookModel" :value.sync="addressbook" :is-first-property="true"
+					:is-last-property="false" :options="addressbooksOptions" class="property--addressbooks" />
 			</section>
 		</template>
 	</div>
diff --git a/src/components/Properties/PropertyGroups.vue b/src/components/Properties/PropertyGroups.vue
index 56a6ce8e..5f126ecc 100644
--- a/src/components/Properties/PropertyGroups.vue
+++ b/src/components/Properties/PropertyGroups.vue
@@ -21,7 +21,7 @@
   -->
 
 <template>
-	<div v-if="propModel" class="grid-span-1 property">
+	<div v-if="propModel" class="grid-span-2 property">
 
 		<div class="property__row">
 			<div class="property__label">{{ propModel.readableName }}</div>
-- 
cgit v1.2.3


From 12ca4f622092cf715a03a0a8d6f9b0c13f9cb9c5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?John=20Molakvo=C3=A6=20=28skjnldsv=29?=
 <skjnldsv@protonmail.com>
Date: Fri, 31 Aug 2018 09:23:57 +0200
Subject: Grid fixes, comments, delete contact and key check
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
---
 src/components/ContactDetails.vue                  | 11 ++++++++++-
 src/components/ContentList/ContentListItem.vue     | 17 +++++++++++++++++
 src/components/Properties/PropertyDateTime.vue     |  1 +
 src/components/Properties/PropertyGroups.vue       |  1 +
 src/components/Properties/PropertyMultipleText.vue |  1 +
 src/components/Properties/PropertySelect.vue       |  1 +
 src/components/Properties/PropertyText.vue         |  1 +
 src/components/Properties/PropertyTitle.vue        |  1 -
 src/store/FakeName.vcf                             |  6 ++++++
 src/store/addressbooks.js                          |  6 ++++++
 10 files changed, 44 insertions(+), 2 deletions(-)

(limited to 'src')

diff --git a/src/components/ContactDetails.vue b/src/components/ContactDetails.vue
index c2bf5d54..a15a6970 100644
--- a/src/components/ContactDetails.vue
+++ b/src/components/ContactDetails.vue
@@ -82,7 +82,8 @@
 			<section class="contact-details">
 
 				<!-- properties iteration -->
-				<contact-details-property v-for="(property, index) in sortedProperties" :key="index" :index="index"
+				<!-- using contact.key in the key and index as key to avoid conflicts between similar data and exact key -->
+				<contact-details-property v-for="(property, index) in sortedProperties" :key="index+contact.key" :index="index"
 					:sorted-properties="sortedProperties" :property="property" :contact="contact"
 					@updatedcontact="updateContact" />
 
@@ -245,6 +246,7 @@ export default {
 		updateContact() {
 			this.$store.dispatch('updateContact', this.contact)
 		},
+
 		/**
 		 * Debounce the contact update for the header props
 		 * photo, fn, org, title
@@ -261,6 +263,13 @@ export default {
 			this.openedMenu = !this.openedMenu
 		},
 
+		/**
+		 * Dispatch contact deletion request
+		 */
+		deleteContact() {
+			this.$store.dispatch('deleteContact', this.contact)
+		},
+
 		/**
 		 * Move contact to the specified addressbook
 		 *
diff --git a/src/components/ContentList/ContentListItem.vue b/src/components/ContentList/ContentListItem.vue
index 227ff912..6c2ccdbf 100644
--- a/src/components/ContentList/ContentListItem.vue
+++ b/src/components/ContentList/ContentListItem.vue
@@ -36,6 +36,11 @@ export default {
 		selectedContact() {
 			return this.$route.params.selectedContact
 		},
+
+		/**
+		 * avatar color based on server toRgb method and the displayName
+		 * @returns {String} the color in css format
+		 */
 		colorAvatar() {
 			try {
 				let color = this.contact.uid.toRgb()
@@ -46,13 +51,25 @@ export default {
 		}
 	},
 	methods: {
+		/**
+		 * Checkbox management method
+		 */
 		toggleSelect() {
 			// toggle checkbox here because we stop the propagation to not trigger selectContact
+			// therefore the selectContact prevent the checkbox label+input propagation
 			this.$refs.selected.checked = !this.$refs.selected.checked
 		},
+
+		/**
+		 * Dispatch contact deletion request
+		 */
 		deleteContact() {
 			this.$store.dispatch('deleteContact', this.contact)
 		},
+
+		/**
+		 * Select this contact within the list
+		 */
 		selectContact() {
 			// change url with router
 			this.$router.push({ name: 'contact', params: { selectedGroup: this.selectedGroup, selectedContact: this.contact.key } })
diff --git a/src/components/Properties/PropertyDateTime.vue b/src/components/Properties/PropertyDateTime.vue
index f4f843bc..74f6ed72 100644
--- a/src/components/Properties/PropertyDateTime.vue
+++ b/src/components/Properties/PropertyDateTime.vue
@@ -106,6 +106,7 @@ export default {
 		/**
 		 * Since we're updating a local data based on the value prop,
 		 * we need to make sure to update the local data on pop change
+		 * TODO: check if this create performance drop
 		 */
 		value: function() {
 			this.localValue = this.value
diff --git a/src/components/Properties/PropertyGroups.vue b/src/components/Properties/PropertyGroups.vue
index 5f126ecc..9979733e 100644
--- a/src/components/Properties/PropertyGroups.vue
+++ b/src/components/Properties/PropertyGroups.vue
@@ -92,6 +92,7 @@ export default {
 		/**
 		 * Since we're updating a local data based on the value prop,
 		 * we need to make sure to update the local data on pop change
+		 * TODO: check if this create performance drop
 		 */
 		value: function() {
 			this.localValue = this.value
diff --git a/src/components/Properties/PropertyMultipleText.vue b/src/components/Properties/PropertyMultipleText.vue
index 45fc70ef..90d2afde 100644
--- a/src/components/Properties/PropertyMultipleText.vue
+++ b/src/components/Properties/PropertyMultipleText.vue
@@ -109,6 +109,7 @@ export default {
 		/**
 		 * Since we're updating a local data based on the value prop,
 		 * we need to make sure to update the local data on pop change
+		 * TODO: check if this create performance drop
 		 */
 		value: function() {
 			this.localValue = this.value
diff --git a/src/components/Properties/PropertySelect.vue b/src/components/Properties/PropertySelect.vue
index 4981eb47..fb1aa827 100644
--- a/src/components/Properties/PropertySelect.vue
+++ b/src/components/Properties/PropertySelect.vue
@@ -111,6 +111,7 @@ export default {
 		/**
 		 * Since we're updating a local data based on the value prop,
 		 * we need to make sure to update the local data on pop change
+		 * TODO: check if this create performance drop
 		 */
 		value: function() {
 			this.localValue = this.value
diff --git a/src/components/Properties/PropertyText.vue b/src/components/Properties/PropertyText.vue
index 32937db0..9cf499c4 100644
--- a/src/components/Properties/PropertyText.vue
+++ b/src/components/Properties/PropertyText.vue
@@ -105,6 +105,7 @@ export default {
 		/**
 		 * Since we're updating a local data based on the value prop,
 		 * we need to make sure to update the local data on pop change
+		 * TODO: check if this create performance drop
 		 */
 		value: function() {
 			this.localValue = this.value
diff --git a/src/components/Properties/PropertyTitle.vue b/src/components/Properties/PropertyTitle.vue
index cde06dd3..360e813a 100644
--- a/src/components/Properties/PropertyTitle.vue
+++ b/src/components/Properties/PropertyTitle.vue
@@ -43,6 +43,5 @@ export default {
 			required: true
 		}
 	}
-
 }
 </script>
diff --git a/src/store/FakeName.vcf b/src/store/FakeName.vcf
index cae71a7b..ebfc383a 100644
--- a/src/store/FakeName.vcf
+++ b/src/store/FakeName.vcf
@@ -9,6 +9,12 @@ REV:2017-07-27T05:56:33Z
 UID:5acf667e-1cbf-48a8-87fe-546ee31a0b23
 END:VCARD
 BEGIN:VCARD
+VERSION:4.0
+KIND:org
+FN:ABC Marketing ORG
+ORG:ABC\, Inc.;North American Division;Marketing
+END:VCARD
+BEGIN:VCARD
 VERSION:3.0
 N:Cunningham;Liam;;Mr.;
 FN:Liam Cunningham
diff --git a/src/store/addressbooks.js b/src/store/addressbooks.js
index 54bd6d8a..b5ce1397 100644
--- a/src/store/addressbooks.js
+++ b/src/store/addressbooks.js
@@ -177,6 +177,12 @@ const actions = {
 			enabled: true,
 			owner: 'admin'
 			// dav: addressbook
+		}, {
+			id: 'ab2',
+			displayName: 'Addressbook 2',
+			enabled: true,
+			owner: 'admin'
+			// dav: addressbook
 		}]
 		// })
 
-- 
cgit v1.2.3