fix: settings icon

Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
This commit is contained in:
Peter Ringelmann 2026-06-01 13:01:12 +02:00 committed by Peter R.
parent 0a36e6a829
commit 352d0cc53b
4 changed files with 16 additions and 5 deletions

View file

@ -57,7 +57,7 @@
<img
class="app-menu__current-app-icon"
:class="{ 'app-menu__current-app-icon--settings': currentApp.type === 'settings' }"
:src="currentApp.icon"
:src="displayIcon"
alt=""
aria-hidden="true">
</template>
@ -172,6 +172,17 @@ export default defineComponent({
: this.currentApp.name
},
// Match the collapsed label: a generic cog for any settings
// sub-section instead of the per-section icon.
displayIcon(): string {
if (!this.currentApp) {
return ''
}
return this.currentApp.type === 'settings'
? imagePath('core', 'actions/settings.svg')
: this.currentApp.icon
},
// aria-label overrides the inner span text, so the displayed name
// has to be duplicated here for screen readers.
currentAppLabel(): string {

4
dist/core-main.js vendored

File diff suppressed because one or more lines are too long

View file

@ -209,7 +209,7 @@ This file is generated from multiple sources. Included packages:
- version: 11.3.0
- license: MIT
- axios
- version: 1.16.1
- version: 1.15.0
- license: MIT
- base64-js
- version: 1.5.1

File diff suppressed because one or more lines are too long