Header icons fixes

Signed-off-by: John Molakvoæ (skjnldsv) <[email protected]>

Author: skjnldsv

css/ContactDetails.scss

@@ -71,22 +71,18 @@
 				position: relative;
 				height: 44px;
 				width: 44px;
+			}
+			.header-icon {
+				height: 44px;
+				width: 44px;
+				padding: 14px;
 				border-radius: 22px;
-				&.menu-icon--pulse {
+				cursor: pointer;
+				&.header-icon--pulse {
 					margin: 8px;
 					width: 16px;
 					height: 16px;
 				}
-
-				&[class^='icon-'],
-				&[class*=' icon-'],
-				> [class^='icon-'],
-				> [class*=' icon-'] {
-					height: 44px;
-					width: 44px;
-					padding: 14px;
-					cursor: pointer;
-				}
 			}
 		}
 	}

src/components/ContactDetails.vue

@@ -67,15 +67,15 @@
 
 				<!-- actions -->
 				<div id="contact-header-actions">
-					<div v-tooltip.bottom="warning" :class="{'icon-loading-small': loadingUpdate, 'menu-icon--pulse icon-error-white': warning}" class="menu-icon" />
+					<div v-tooltip.bottom="warning" :class="{'icon-loading-small': loadingUpdate, 'header-icon--pulse icon-error-white': warning}" class="header-icon" />
 					<div v-tooltip="{
 							content: conflict,
 							show: true,
 							trigger: 'manual',
-						}" v-if="conflict" class="menu-icon menu-icon--pulse icon-history-white"
+						}" v-if="conflict" class="header-icon header-icon--pulse icon-history-white"
 						@click="refreshContact" />
 					<div class="menu-icon">
-						<div v-click-outside="closeMenu" class="icon-more-white" @click="toggleMenu" />
+						<div v-click-outside="closeMenu" class="header-icon icon-more-white" @click="toggleMenu" />
 						<div :class="{ 'open': openedMenu }" class="popovermenu">
 							<popover-menu :menu="contactActions" />
 						</div>

src/views/Contacts.vue

@@ -111,7 +111,7 @@ export default {
 		},
 		// first enabled addressbook of the list
 		defaultAddressbook() {
-			return this.addressbooks.find(addressbook => addressbook.readOnly !== false)
+			return this.addressbooks.find(addressbook => !addressbook.readOnly)
 		},
 
 		/**