Refactor icon rendering

Author: twharmon

frameworks/keyed/rendrjs/package-lock.json

@@ -10,7 +10,7 @@
     "webpack-cli": "5.1.4"
   },
   "dependencies": {
-    "@rendrjs/core": "0.1.202"
+    "@rendrjs/core": "0.1.205"
   },
   "js-framework-benchmark": {
     "frameworkVersionFromPackage": "@rendrjs/core",

frameworks/keyed/rendrjs/package.json

@@ -1,7 +1,7 @@
 import { rendr, useState } from '@rendrjs/core';
 import { Jumbotron } from './Jumbotron';
 import { Row } from './Row';
-import { RemoveIcon } from './RemoveIcon';
+import { makeIcon } from './RemoveIcon';
 
 let random = (max) => Math.round(Math.random() * 1000) % max;
 
@@ -29,7 +29,7 @@ let buildData = (count) => {
   return data;
 };
 
-export let preloadIcon = rendr(RemoveIcon, { pre: true });
+export let preloadIcon = makeIcon(true);
 
 export let App = () => {
   let [state, setState] = useState({ arr: [], sel: 0 });

frameworks/keyed/rendrjs/src/App.js

@@ -1,6 +1,6 @@
 import { rendr } from '@rendrjs/core';
 
-export let RemoveIcon = ({ pre }) => rendr('span', {
-    class: `${pre ? 'preloadicon ' : ''}glyphicon glyphicon-remove`,
-    'aria-hidden': true,
+export let makeIcon = preload => rendr('span', {
+    class: `${preload ? 'preloadicon ' : ''}glyphicon glyphicon-remove`,
+    ariaHidden: true,
 });

frameworks/keyed/rendrjs/src/RemoveIcon.js

@@ -1,6 +1,8 @@
 import { rendr } from '@rendrjs/core';
 import { colMd1, colMd4, colMd6 } from './classes';
-import { RemoveIcon } from './RemoveIcon';
+import { makeIcon } from './RemoveIcon';
+
+let icon = makeIcon();
 
 export let Row = ({ hi, sel, item, del }) => {
   return rendr('tr', {
@@ -21,7 +23,7 @@ export let Row = ({ hi, sel, item, del }) => {
         class: colMd1,
         slot: rendr('a', {
           onclick: () => del(item.id),
-          slot: rendr(RemoveIcon, {}),
+          slot: icon,
         }),
       }),
       rendr('td', { class: colMd6 }),