Merge pull request #910 from vorth/zometool-products

Adjustments to zometool products list
vZome · Aug 19, 2024 · c81d9d6 · c81d9d6
2 parents f6f9f00 + e0aabdc
commit c81d9d6
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/online/serve/app/test/cases/zometool-model/index.html b/online/serve/app/test/cases/zometool-model/index.html
@@ -31,7 +31,7 @@ <h1>Zometool Model Demo</h1>
       </zometool-instructions>      
 
       <zometool-parts-required></zometool-parts-required>
-      <zometool-covering-products>Supporting Products:</zometool-covering-products>
+      <zometool-covering-products></zometool-covering-products>
 
     </div>
     <!-- ^^^^^^^ ----- This is the portion you will enter as content HTML for the page -->

diff --git a/online/src/wc/zometool/products-list.jsx b/online/src/wc/zometool/products-list.jsx
@@ -21,7 +21,7 @@ const findProducts = ( bom, parts, products ) =>
 
   const result = [];
   Object.entries( products ) .map( ([ code, product ]) => {
-    if ( productCoversBom( product, bom ) )
+    if ( product.url && productCoversBom( product, bom ) ) // ignore products without a URL
       result .push( { name: product.label, url: product.url } );
   } );
   return result;

diff --git a/online/src/wc/zometool/zometool.css.js b/online/src/wc/zometool/zometool.css.js
@@ -105,6 +105,10 @@ export const instructionsCSS = `
   grid-template-rows: min-content 1fr 90px;
 }
 
+zometool-covering-products::before {
+   content: "Supporting products:";
+}
+
 .matched-product {
   min-width: 250px;
   display: flex;