circles
circles copied to clipboard
Change fill to white to match other icons.
Fixes #703
The dark icon would still be needed, so maybe you can put it as circles-dark.svg and make sure that it is used in the places where the current one was used before:
lib/Notification/Notifier.php 150: $iconPath = $this->urlGenerator->imagePath(Application::APP_ID, 'circles.svg');
lib/Activity/Filter.php 54: $this->url->imagePath(Application::APP_ID, 'circles.svg')
lib/UnifiedSearch/UnifiedSearchProvider.php 163: $iconPath = $this->urlGenerator->imagePath(Application::APP_ID, 'circles.svg');
lib/Model/ShareWrapper.php 688: $urlGenerator->imagePath(Application::APP_ID, 'circles.svg')
@juliushaertl Hi!
A separate icon for a dark theme is not required; the dark theme expects a pure white filled icon which it then styles:
This behavior is consistent with other 1st party apps in Nextcloud.
It is not about the dark theme but other places where the icon is used like notifications or activity. There a dark icon variant would still be required.
Done in another PR