- ๐บ๐ธUnited States bnjmnm Ann Arbor, MI
When I filed this issue in 2020...
- Claro was an experimental theme. It has since become Drupal's admin theme
- I was not a frontend framework manager. I have since become a frontend framework manager
Given that it is 2+ years since Claro graduated from experimental status, I'm not sure the benefits of of moving the assets + references outweigh the potential disruption. Testing this manually requires time and it is a lengthy process without tests to confirm things are OK. Something unwanted could slip through, and custom modules/themes may be referencing some of these assets. Had this happened while Claro was still experimental I'd be all for the change. While it should have occurred when Claro was added to core on October 4, 2019, that doesn't necessarily mean it should happen now. I'm setting this to Closed (outdated), but I don't intend this to be a stern "final word". If anyone here believes the benefits of doing this outweigh the risks then document that and re-open.
Alternatively, get some quick issue credit and open an issue that simply removes this from the readme:
Icons in this folder are copied from Drupal core. This folder with its content should be removed before moving Claro to Drupal core