feat: make profile picture clickable (#1531)

Co-authored-by: Dominik Pschenitschni <mail@celement.de>
Reviewed-on: https://kolaente.dev/vikunja/frontend/pulls/1531
Reviewed-by: konrad <k@knt.li>
Co-authored-by: Dominik Pschenitschni <dpschen@noreply.kolaente.de>
Co-committed-by: Dominik Pschenitschni <dpschen@noreply.kolaente.de>

cypress/integration/user/logout.spec.js

@@ -6,7 +6,7 @@ describe('Log out', () => {
 
 		cy.get('.navbar .user .username')
 			.click()
-		cy.get('.navbar .user .dropdown-menu a.dropdown-item')
+		cy.get('.navbar .user .dropdown-menu .dropdown-item')
 			.contains('Logout')
 			.click()
 

src/App.vue

@@ -1,7 +1,7 @@
 <template>
 	<ready>
 		<template v-if="authUser">
-			<top-navigation/>
+			<TheNavigation/>
 			<content-auth/>
 		</template>
 		<content-link-share v-else-if="authLinkShare"/>
@@ -27,7 +27,7 @@ import {success} from '@/message'
 
 import Notification from '@/components/misc/notification.vue'
 import KeyboardShortcuts from './components/misc/keyboard-shortcuts/index.vue'
-import TopNavigation from './components/home/topNavigation.vue'
+import TheNavigation from '@/components/home/TheNavigation.vue'
 import ContentAuth from './components/home/contentAuth.vue'
 import ContentLinkShare from './components/home/contentLinkShare.vue'
 import NoAuthWrapper from '@/components/misc/no-auth-wrapper.vue'

src/components/home/TheNavigation.vue

@@ -32,12 +32,13 @@
 			</a>
 			<notifications/>
 			<div class="user">
-				<img :src="userAvatar" alt="" class="avatar" width="40" height="40"/>
 				<dropdown class="is-right" ref="usernameDropdown">
 					<template #trigger>
 						<x-button
 							variant="secondary"
-							:shadow="false">
+							:shadow="false"
+						>
+							<img :src="userAvatar" alt="" class="avatar" width="40" height="40"/>
 							<span class="username">{{ userInfo.name !== '' ? userInfo.name : userInfo.username }}</span>
 							<span class="icon is-small">
 								<icon icon="chevron-down"/>
@@ -45,92 +46,96 @@
 						</x-button>
 					</template>
 
-					<router-link :to="{name: 'user.settings'}" class="dropdown-item">
+					<BaseButton
+						:to="{name: 'user.settings'}"
+						class="dropdown-item"
+					>
 						{{ $t('user.settings.title') }}
-					</router-link>
+					</BaseButton>
-					<a
+					<BaseButton
+						v-if="imprintUrl"
 						:href="imprintUrl"
 						class="dropdown-item"
-						target="_blank"
+					>
-						rel="noreferrer noopener nofollow"
-						v-if="imprintUrl">
 						{{ $t('navigation.imprint') }}
-					</a>
+					</BaseButton>
-					<a
+					<BaseButton
+						v-if="privacyPolicyUrl"
 						:href="privacyPolicyUrl"
 						class="dropdown-item"
-						target="_blank"
+					>
-						rel="noreferrer noopener nofollow"
-						v-if="privacyPolicyUrl">
 						{{ $t('navigation.privacy') }}
-					</a>
+					</BaseButton>
-					<a @click="$store.commit('keyboardShortcutsActive', true)" class="dropdown-item">
+					<BaseButton
+						@click="$store.commit('keyboardShortcutsActive', true)"
+						class="dropdown-item"
+					>
 						{{ $t('keyboardShortcuts.title') }}
-					</a>
+					</BaseButton>
-					<router-link :to="{name: 'about'}" class="dropdown-item">
+					<BaseButton
+						:to="{name: 'about'}"
+						class="dropdown-item"
+					>
 						{{ $t('about.title') }}
-					</router-link>
+					</BaseButton>
-					<a @click="logout()" class="dropdown-item">
+					<BaseButton
+						@click="logout()"
+						class="dropdown-item"
+					>
 						{{ $t('user.auth.logout') }}
-					</a>
+					</BaseButton>
 				</dropdown>
 			</div>
 		</div>
 	</header>
 </template>
 
-<script>
+<script setup langs="ts">
-import {mapState} from 'vuex'
+import {ref, computed, onMounted, nextTick} from 'vue'
-import {CURRENT_LIST, QUICK_ACTIONS_ACTIVE} from '@/store/mutation-types'
+import {useStore} from 'vuex'
+import {useRouter} from 'vue-router'
+
+import {QUICK_ACTIONS_ACTIVE} from '@/store/mutation-types'
 import Rights from '@/models/constants/rights.json'
+
 import Update from '@/components/home/update.vue'
 import ListSettingsDropdown from '@/components/list/list-settings-dropdown.vue'
 import Dropdown from '@/components/misc/dropdown.vue'
 import Notifications from '@/components/notifications/notifications.vue'
 import Logo from '@/components/home/Logo.vue'
+import BaseButton from '@/components/base/BaseButton.vue'
 import MenuButton from '@/components/home/MenuButton.vue'
 
-export default {
+const store = useStore()
-	name: 'topNavigation',
+
-	components: {
+const userInfo = computed(() => store.state.auth.info)
-		Notifications,
+const userAvatar = computed(() => store.state.auth.avatarUrl)
-		Dropdown,
+const currentList = computed(() => store.state.currentList)
-		ListSettingsDropdown,
+const background = computed(() => store.state.background)
-		Update,
+const imprintUrl = computed(() => store.state.config.legal.imprintUrl)
-		Logo,
+const privacyPolicyUrl = computed(() => store.state.config.legal.privacyPolicyUrl)
-		MenuButton,
+const canWriteCurrentList = computed(() => store.state.currentList.maxRight > Rights.READ)
-	},
+
-	computed: {
+const usernameDropdown = ref()
-		...mapState({
+const listTitle = ref()
-			userInfo: state => state.auth.info,
+onMounted(async () => {
-			userAvatar: state => state.auth.avatarUrl,
+	await nextTick()
-			userAuthenticated: state => state.auth.authenticated,
+	if (typeof usernameDropdown.value === 'undefined' || typeof listTitle.value === 'undefined') {
-			currentList: CURRENT_LIST,
-			background: 'background',
-			imprintUrl: state => state.config.legal.imprintUrl,
-			privacyPolicyUrl: state => state.config.legal.privacyPolicyUrl,
-			canWriteCurrentList: state => state.currentList.maxRight > Rights.READ,
-		}),
-	},
-	mounted() {
-		this.$nextTick(() => {
-			if (typeof this.$refs.usernameDropdown === 'undefined' || typeof this.$refs.listTitle === 'undefined') {
 		return
 	}
 
-			const usernameWidth = this.$refs.usernameDropdown.$el.clientWidth
+	const usernameWidth = usernameDropdown.value.$el.clientWidth
-			this.$refs.listTitle.style.setProperty('--nav-username-width', `${usernameWidth}px`)
+	listTitle.value.style.setProperty('--nav-username-width', `${usernameWidth}px`)
-		})
+})
-	},
+
-	methods: {
+const router = useRouter()
-		logout() {
+function logout() {
-			this.$store.dispatch('auth/logout')
+	store.dispatch('auth/logout')
-			this.$router.push({name: 'user.login'})
+	router.push({name: 'user.login'})
-		},
+}
-		openQuickActions() {
+
-			this.$store.commit(QUICK_ACTIONS_ACTIVE, true)
+function openQuickActions() {
-		},
+	store.commit(QUICK_ACTIONS_ACTIVE, true)
-	},
 }
 </script>
 
@@ -246,6 +251,7 @@ $hamburger-menu-icon-width: 28px;
 			border-radius: 100%;
 			vertical-align: middle;
 			height: 40px;
+			margin-right: var(--button-padding-horizontal);
 		}
 
 		:deep(.dropdown-trigger .button) {