From f940538d377b3113c617a45590cd759f01def786 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Raimund=20Schl=C3=BC=C3=9Fler?=
 <raimund.schluessler@mailbox.org>
Date: Wed, 22 Dec 2021 21:58:27 +0100
Subject: [PATCH] Move crumb Actions into hover background area
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Raimund Schlüßler <raimund.schluessler@mailbox.org>
---
 src/components/Breadcrumb/Breadcrumb.vue | 83 ++++++++++++++----------
 1 file changed, 48 insertions(+), 35 deletions(-)

diff --git a/src/components/Breadcrumb/Breadcrumb.vue b/src/components/Breadcrumb/Breadcrumb.vue
index f016a83b06..297ea35640 100644
--- a/src/components/Breadcrumb/Breadcrumb.vue
+++ b/src/components/Breadcrumb/Breadcrumb.vue
@@ -31,7 +31,11 @@ This component is meant to be used inside a Breadcrumbs component.
 <template>
 	<div ref="crumb"
 		class="vue-crumb"
-		:class="{'vue-crumb--with-action': $slots.default, 'vue-crumb--hovered': hovering}"
+		:class="{
+			'vue-crumb--with-action': $slots.default,
+			'vue-crumb--only-action': (!($slots.icon || icon || title)),
+			'vue-crumb--hovered': hovering
+		}"
 		:[crumbId]="''"
 		draggable="false"
 		@dragstart.prevent="() => {/** Prevent the breadcrumb from being draggable. */}"
@@ -39,29 +43,32 @@ This component is meant to be used inside a Breadcrumbs component.
 		@dragover.prevent="() => {}"
 		@dragenter="dragEnter"
 		@dragleave="dragLeave">
-		<element
-			:is="tag"
-			v-if="title || icon"
-			:to="to"
-			:href="href">
-			<!-- @slot Slot for passing a material design icon. Precedes the icon and title prop. -->
-			<slot name="icon">
-				<span v-if="icon" :class="icon" class="icon" />
-				<span v-else>{{ title }}</span>
-			</slot>
-		</element>
-		<Actions ref="actions"
-			:force-menu="forceMenu"
-			:open="open"
-			:container="`.vue-crumb--with-action[${crumbId}]`"
-			@update:open="onOpenChange">
-			<template #icon>
-				<!-- @slot Slot for the custom menu icon -->
-				<slot name="menu-icon" />
-			</template>
-			<!-- @slot All action elements passed into the default slot will be used -->
-			<slot />
-		</Actions>
+		<div>
+			<element
+				:is="tag"
+				v-if="(title || icon) && !$slots.default"
+				:to="to"
+				:href="href">
+				<!-- @slot Slot for passing a material design icon. Precedes the icon and title prop. -->
+				<slot name="icon">
+					<span v-if="icon" :class="icon" class="icon" />
+					<span v-else>{{ title }}</span>
+				</slot>
+			</element>
+			<Actions ref="actions"
+				:force-menu="forceMenu"
+				:open="open"
+				:menu-title="title"
+				:container="`.vue-crumb--with-action[${crumbId}]`"
+				@update:open="onOpenChange">
+				<template #icon>
+					<!-- @slot Slot for the custom menu icon -->
+					<slot name="menu-icon" />
+				</template>
+				<!-- @slot All action elements passed into the default slot will be used -->
+				<slot />
+			</Actions>
+		</div>
 		<ChevronRight class="vue-crumb__separator" :size="20" />
 	</div>
 </template>
@@ -239,10 +246,8 @@ export default {
 
 	&:last-child {
 		max-width: 210px;
+		font-weight: bold;
 
-		a {
-			flex-shrink: 1;
-		}
 		// Don't show breadcrumb separator for last crumb
 		.vue-crumb__separator {
 			display: none;
@@ -253,37 +258,45 @@ export default {
 		display: none;
 	}
 
-	&--with-action a {
-		padding-right: 2px;
+	&#{&}--with-action > div {
+		padding-right: 0;
 	}
 
-	&__separator {
+	&#{&}--only-action > div {
 		padding: 0;
-		opacity: .7;
 	}
 
-	&#{&}--hovered > a {
+	&#{&}--hovered > div {
 		background-color: var(--color-background-dark);
 		opacity: .7;
 	}
 
-	> a {
+	&__separator {
+		padding: 0;
+		opacity: .7;
+	}
+
+	> div {
 		&:hover,
-		&:focus {
+		&:focus-within {
 			background-color: var(--color-background-dark);
 		}
-		&:focus{
+		&:focus-within {
 			opacity: 1;
 		}
 		&:hover {
 			opacity: .7;
 		}
+		> a {
+			overflow: hidden;
+		}
 		opacity: .5;
 		padding: 12px;
 		max-width: 100%;
 		border-radius: var(--border-radius-pill);
 	}
 
+	> div,
 	a {
 		align-items: center;
 		display: inline-flex;