Remove the fill attribute from all icons (#488)

The fill attribute prevents the icon from receiving the proper coloring. This wasn't an issue in the symbolset version because svg-symbols strips the attribute.

Author: Windvis

lib/generate-icon-components.mjs

@@ -15,6 +15,8 @@ const COMPONENT_ICON_FOLDER = join(
   "icons",
 );
 
+const iconsWithFill = [];
+
 const files = await readdir(ICON_FOLDER);
 const icons = files
   .filter((file) => extname(file) === ".svg")
@@ -27,6 +29,12 @@ const promises = icons.map((svg) => {
 });
 await Promise.all(promises);
 
+if (iconsWithFill.length > 0) {
+  throw new Error(
+    `The following icons have a fill attribute which might cause issues: ${iconsWithFill.join(", ")}`,
+  );
+}
+
 async function generateComponent(iconName) {
   const componentName = pascalCase(iconName, {
     mergeAmbiguousCharacters: true,
@@ -36,6 +44,10 @@ async function generateComponent(iconName) {
     .toString()
     .replace(">", " ...attributes>"); // We assume the first closing bracket belongs to the svg element
 
+  if (iconContent.includes('fill="')) {
+    iconsWithFill.push(iconName);
+  }
+
   const componentContent = `// THIS FILE IS GENERATED. ANY CHANGES TO THIS FILE WILL BE LOST.
 import type { TOC } from '@ember/component/template-only';