Skip to content

Commit b7dda0c

Browse files
pringelmannbackportbot[bot]
authored andcommitted
fix: settings icon
Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
1 parent 954839f commit b7dda0c

1 file changed

Lines changed: 12 additions & 1 deletion

File tree

core/src/components/AppMenu.vue

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@
5757
<img
5858
class="app-menu__current-app-icon"
5959
:class="{ 'app-menu__current-app-icon--settings': currentApp.type === 'settings' }"
60-
:src="currentApp.icon"
60+
:src="displayIcon"
6161
alt=""
6262
aria-hidden="true">
6363
</template>
@@ -172,6 +172,17 @@ export default defineComponent({
172172
: this.currentApp.name
173173
},
174174
175+
// Match the collapsed label: a generic cog for any settings
176+
// sub-section instead of the per-section icon.
177+
displayIcon(): string {
178+
if (!this.currentApp) {
179+
return ''
180+
}
181+
return this.currentApp.type === 'settings'
182+
? imagePath('core', 'actions/settings.svg')
183+
: this.currentApp.icon
184+
},
185+
175186
// aria-label overrides the inner span text, so the displayed name
176187
// has to be duplicated here for screen readers.
177188
currentAppLabel(): string {

0 commit comments

Comments
 (0)