Refactor code: use .append() for adding multiple elements

googlefonts · Feb 6, 2025 · 2f17509 · 2f17509
1 parent d6ef3ee
commit 2f17509
Showing 1 changed file with 4 additions and 10 deletions.
diff --git a/src/fontra/views/fontinfo/panel-sources.js b/src/fontra/views/fontinfo/panel-sources.js
@@ -683,28 +683,22 @@ class SourceBox extends HTMLElement {
     this.append(
       html.div({ class: "fontra-ui-font-info-sources-panel-header" }, [
         getLabelFromKey("general"),
-      ])
+      ]),
+      buildElement(this.controllers.general)
     );
-    this.append(buildElement(this.controllers.general));
-
     // Don't add 'Location', if the font has no axes.
     if (this.fontAxesSourceSpace.length > 0) {
       this.append(
         html.div({ class: "fontra-ui-font-info-sources-panel-header" }, [
           getLabelFromKey("location"),
-        ])
-      );
-      this.append(
+        ]),
         buildElementLocations(this.controllers.location, this.fontAxesSourceSpace)
       );
     }
-
     this.append(
       html.div({ class: "fontra-ui-font-info-sources-panel-header" }, [
         getLabelFromKey("lineMetricsHorizontalLayout"),
-      ])
-    );
-    this.append(
+      ]),
       buildElementLineMetricsHor(this.controllers.lineMetricsHorizontalLayout)
     );
   }