Because Drupal\ui_icons\Plugin\IconPackManager is in the scope of
✨
Add an icon management API
Active
And because we don't want to merge any "UI Icons" branding in Core. Let's rename the render element from *.ui_icons.yml to *.icons.yml.
Rename, check where all the places where it is used (including tests):
protected function getDiscovery(): DiscoveryInterface {
if (!$this->discovery) {
$this->discovery = new YamlDiscovery('ui_icons', $this->moduleHandler->getModuleDirectories() + $this->themeHandler->getThemeDirectories());
$this->discovery = new ContainerDerivativeDiscoveryDecorator($this->discovery);
}
return $this->discovery;
}
Is there other "ui icons" stuff we need to rename before the MR?
Active
1.0
Code
Issue #3480910 by mogtofu33, pdureau: Rename *.ui_icons.yml to *.icons....
Automatically closed - issue fixed for 2 weeks with no activity.