aboutsummaryrefslogtreecommitdiffstats
path: root/lib
diff options
context:
space:
mode:
authorJoas Schilling <213943+nickvergessen@users.noreply.github.com>2024-03-08 17:07:06 +0100
committerGitHub <noreply@github.com>2024-03-08 17:07:06 +0100
commitb2b4f34d967e1aed556847b349b8b39f429938d2 (patch)
tree884e3e321ee63ecab3cc64f346340fcc8778c7c8 /lib
parent170f93ad725c31f0ac870ac91e5ba78617c1c2cf (diff)
parentc8d7a5acaabb5c2f8bae1ccb70ce9c76cfd86c5c (diff)
downloadnextcloud-server-b2b4f34d967e1aed556847b349b8b39f429938d2.tar.gz
nextcloud-server-b2b4f34d967e1aed556847b349b8b39f429938d2.zip
Merge pull request #44073 from nextcloud/fix/provide-correct-icon
fix(AppManager): Allow to query dark **or** bright icon
Diffstat (limited to 'lib')
-rw-r--r--lib/private/App/AppManager.php4
-rw-r--r--lib/public/App/IAppManager.php3
2 files changed, 4 insertions, 3 deletions
diff --git a/lib/private/App/AppManager.php b/lib/private/App/AppManager.php
index 5dec4e7ccde..c6e6ba2975e 100644
--- a/lib/private/App/AppManager.php
+++ b/lib/private/App/AppManager.php
@@ -109,8 +109,8 @@ class AppManager implements IAppManager {
) {
}
- public function getAppIcon(string $appId): ?string {
- $possibleIcons = [$appId . '.svg', 'app.svg', $appId . '-dark.svg', 'app-dark.svg'];
+ public function getAppIcon(string $appId, bool $dark = false): ?string {
+ $possibleIcons = $dark ? [$appId . '-dark.svg', 'app-dark.svg'] : [$appId . '.svg', 'app.svg'];
$icon = null;
foreach ($possibleIcons as $iconName) {
try {
diff --git a/lib/public/App/IAppManager.php b/lib/public/App/IAppManager.php
index 968771388dc..142e8bb8515 100644
--- a/lib/public/App/IAppManager.php
+++ b/lib/public/App/IAppManager.php
@@ -65,10 +65,11 @@ interface IAppManager {
* Returns the app icon or null if none is found
*
* @param string $appId
+ * @param bool $dark Enable to request a dark icon variant, default is a white icon
* @return string|null
* @since 29.0.0
*/
- public function getAppIcon(string $appId): string|null;
+ public function getAppIcon(string $appId, bool $dark = false): string|null;
/**
* Check if an app is enabled for user