Refactor loadContacts() function

astik-dev · Apr 24, 2024 · 1427a1a · 1427a1a
1 parent cebbb11
commit 1427a1a
Showing 1 changed file with 20 additions and 12 deletions.
diff --git a/src/js/modules/dataLoaders/loadContacts.js b/src/js/modules/dataLoaders/loadContacts.js
@@ -3,23 +3,31 @@ import { removeEmptyItems } from "./removeEmptyItems.js";
 import imageCreator from "../imageCreator.js";
 
 
-export function loadContacts(contacts) {
-	const contactsItems = dqs(".contacts__items");
+function contactsItemHTML({link, title, img, imgWEBP}) {
 
-	removeEmptyItems(".contacts__item_empty", contactsItems);
+    const fallback = `contacts/${img}`;
 
-	contacts.forEach((contact) => {
+	const imgElem = imgWEBP
+        ? imageCreator.newWebpPic("external", `contacts/${imgWEBP}`, fallback, title)
+        : imageCreator.newImg("external", fallback, title);
 
-		const fallback = `contacts/${contact.img}`, alt = contact.title;
+    return `
+        <a href="${link}" target="_blank" title="${title}" class="contacts__item">
+            ${imgElem}
+        </a>
+    `;
+}
 
-		const imgElem = contact.imgWEBP ?
-			imageCreator.newWebpPic("external", `contacts/${contact.imgWEBP}`, fallback, alt) :
-			imageCreator.newImg("external", fallback, alt);
+export function loadContacts(contacts) {
+
+    const contactsItems = dqs(".contacts__items");
 
-		let currentContact = `<a href="${contact.link}" target="_blank" title="${contact.title}" class="contacts__item">
-								  ${imgElem}
-							  </a>`
+	removeEmptyItems(".contacts__item_empty", contactsItems);
 
-		contactsItems.insertAdjacentHTML("beforeend", currentContact);
+	contacts.forEach((contact) => {
+		contactsItems.insertAdjacentHTML(
+            "beforeend",
+            contactsItemHTML(contact)
+        );
 	});
 }