Skip to content

Commit

Permalink
Merge pull request #42507 from nextcloud/fix/files/42460/breadcrumb-c…
Browse files Browse the repository at this point in the history
…onditional-rendering

Fix/files/42460/breadcrumb conditional rendering
  • Loading branch information
emoral435 authored Jan 5, 2024
2 parents ac10ea4 + 2554419 commit 2e87874
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 6 deletions.
16 changes: 13 additions & 3 deletions apps/files/src/components/BreadCrumbs.vue
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,8 @@
v-bind="section"
dir="auto"
:to="section.to"
:title="titleForSection(section)"
:title="titleForSection(index, section)"
:aria-description="ariaForSection(index, section)"
@click.native="onClick(section.to)">
<template v-if="index === 0" #icon>
<Home :size="20"/>
Expand Down Expand Up @@ -132,11 +133,20 @@ export default defineComponent({
}
},

titleForSection(section) {
titleForSection(index, section) {
if (section?.to?.query?.dir === this.$route.query.dir) {
return t('files', 'Reload current directory')
} else if (index === 0) {
return t('files', 'Go to the "{dir}" directory', section)
}
return t('files', 'Go to the "{dir}" directory', section)
return null
},

ariaForSection(index, section) {
if (index === section.length - 1) {
return t('files', 'Reload current directory')
}
return null
},

t,
Expand Down
4 changes: 2 additions & 2 deletions dist/files-main.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/files-main.js.map

Large diffs are not rendered by default.

0 comments on commit 2e87874

Please sign in to comment.