From 653d215754c9eb3ffe15db925219a181a7213c52 Mon Sep 17 00:00:00 2001
From: Soham Shinde <65284576+soham04@users.noreply.github.com>
Date: Tue, 28 Oct 2025 04:09:52 +0000
Subject: [PATCH 1/2] fix: Make current folder non-clickable in breadcrumb path
 bar

- Add isCurrent flag to identify last breadcrumb item
- Disable 'to' prop and click handler for current directory
- Current folder now appears as a label instead of clickable button
- Fixes visual spacing issue with the New button

Fixes #55988
---
 apps/files/src/components/BreadCrumbs.vue | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/apps/files/src/components/BreadCrumbs.vue b/apps/files/src/components/BreadCrumbs.vue
index 36cfc58f45f84..307b12ec317f6 100644
--- a/apps/files/src/components/BreadCrumbs.vue
+++ b/apps/files/src/components/BreadCrumbs.vue
@@ -19,7 +19,7 @@
 			:force-icon-text="index === 0 && fileListWidth >= 486"
 			:title="titleForSection(index, section)"
 			:aria-description="ariaForSection(section)"
-			@click.native="onClick(section.to)"
+			@click.native="section.isCurrent ? undefined : onClick(section.to)"
 			@dragover.native="onDragOver($event, section.dir)"
 			@drop="onDrop($event, section.dir)">
 			<template v-if="index === 0" #icon>
@@ -110,13 +110,15 @@ export default defineComponent({
 			return this.dirs.map((dir: string, index: number) => {
 				const source = this.getFileSourceFromPath(dir)
 				const node: Node | undefined = source ? this.getNodeFromSource(source) : undefined
+				const isCurrent = index === this.dirs.length - 1
 				return {
-					dir,
-					exact: true,
-					name: this.getDirDisplayName(dir),
-					to: this.getTo(dir, node),
-					// disable drop on current directory
-					disableDrop: index === this.dirs.length - 1,
+				dir,
+				exact: true,
+				name: this.getDirDisplayName(dir),
+				to: isCurrent ? undefined : this.getTo(dir, node),
+				// disable drop on current directory
+				disableDrop: isCurrent,
+				isCurrent,
 				}
 			})
 		},

From 54ac4c5a02f64da10f43882de12e8a38a43095c4 Mon Sep 17 00:00:00 2001
From: Soham Shinde <65284576+soham04@users.noreply.github.com>
Date: Tue, 28 Oct 2025 04:29:35 +0000
Subject: [PATCH 2/2] Fix: accessibility, tabindex and styling for current
 breadcrumb; update aria/title

---
 apps/files/src/components/BreadCrumbs.vue | 32 ++++++++++++++++++++---
 1 file changed, 29 insertions(+), 3 deletions(-)

diff --git a/apps/files/src/components/BreadCrumbs.vue b/apps/files/src/components/BreadCrumbs.vue
index 307b12ec317f6..db944b4a77294 100644
--- a/apps/files/src/components/BreadCrumbs.vue
+++ b/apps/files/src/components/BreadCrumbs.vue
@@ -16,19 +16,23 @@
 			v-bind="section"
 			dir="auto"
 			:to="section.to"
+			:class="{ 'is-current': section.isCurrent }"
 			:force-icon-text="index === 0 && fileListWidth >= 486"
 			:title="titleForSection(index, section)"
 			:aria-description="ariaForSection(section)"
-			@click.native="section.isCurrent ? undefined : onClick(section.to)"
+			:aria-current="section.isCurrent ? 'page' : null"
+			:tabindex="section.isCurrent ? -1 : 0"
+			@click.native="onClickIfNotCurrent(section.to, index)"
 			@dragover.native="onDragOver($event, section.dir)"
 			@drop="onDrop($event, section.dir)">
 			<template v-if="index === 0" #icon>
 				<NcIconSvgWrapper
-					:size="20"
-					:svg="viewIcon" />
+				:size="20"
+				:svg="viewIcon" />
 			</template>
 		</NcBreadcrumb>
 
+
 		<!-- Forward the actions slot -->
 		<template #actions>
 			<slot name="actions" />
@@ -149,6 +153,14 @@ export default defineComponent({
 	},
 
 	methods: {
+
+		// Guarded click helper — prevents navigation/click for the current (last) breadcrumb
+		onClickIfNotCurrent(to, index) {
+			// if sections isn't defined for some reason, fallback to not navigating
+			if (!this.sections || index === this.sections.length - 1) return
+			this.onClick(to)
+		},
+
 		getNodeFromSource(source: FileSource): Node | undefined {
 			return this.filesStore.getNode(source)
 		},
@@ -272,6 +284,9 @@ export default defineComponent({
 		},
 
 		titleForSection(index, section) {
+			if (section.isCurrent) {
+				return t('files', 'Current directory')
+			}
 			if (section?.to?.query?.dir === this.$route.query.dir) {
 				return t('files', 'Reload current directory')
 			} else if (index === 0) {
@@ -306,8 +321,19 @@ export default defineComponent({
 		a {
 			cursor: pointer !important;
 		}
+
+		/* When the breadcrumb item is the current one, make it non-interactive & not look like a link */
+		.is-current {
+			cursor: default !important;
+		}
+		.is-current a {
+			pointer-events: none !important;
+			text-decoration: none !important;
+			cursor: default !important;
+		}
 	}
 
+
 	&--with-progress {
 		flex-direction: column !important;
 		align-items: flex-start !important;