Skip to content

Commit

Permalink
Merge pull request #41432 from nextcloud/fix/white-user-icon-search
Browse files Browse the repository at this point in the history
Fix search providers icons
  • Loading branch information
Altahrim authored Nov 14, 2023
2 parents 93ea634 + 79d1e40 commit e90df64
Show file tree
Hide file tree
Showing 5 changed files with 10 additions and 1 deletion.
1 change: 1 addition & 0 deletions apps/settings/img/settings.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
1 change: 1 addition & 0 deletions apps/settings/img/settings_apps.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
1 change: 1 addition & 0 deletions apps/settings/img/users-white.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
2 changes: 1 addition & 1 deletion apps/settings/img/users.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
6 changes: 6 additions & 0 deletions lib/private/Search/SearchComposer.php
Original file line number Diff line number Diff line change
Expand Up @@ -223,6 +223,12 @@ private function fetchIcon(string $appId, string $providerId): string {
[$appId, 'app.svg'],
['core', 'places/default-app-icon.svg'],
];
if ($appId === 'settings' && $providerId === 'users') {
// Conflict:
// the file /apps/settings/users.svg is already used in black version by top right user menu
// Override icon name here
$icons = [['settings', 'users-white.svg']];
}
foreach ($icons as $i => $icon) {
try {
return $this->urlGenerator->imagePath(... $icon);
Expand Down

0 comments on commit e90df64

Please sign in to comment.