From af54b890a7e4d8795e4a9e49145bb9b0707f22e3 Mon Sep 17 00:00:00 2001
From: MaxOvs <rock_maksimus@mail.ru>
Date: Tue, 4 Oct 2022 21:53:31 +0300
Subject: [PATCH] Multiselect in working, but requires refactor!

---
 src/cg-dropdown.js | 53 +++++++++++++++++++++++-----------------------
 src/index.js       |  4 ++--
 2 files changed, 28 insertions(+), 29 deletions(-)

diff --git a/src/cg-dropdown.js b/src/cg-dropdown.js
index 5eeee57..a858828 100644
--- a/src/cg-dropdown.js
+++ b/src/cg-dropdown.js
@@ -240,29 +240,14 @@ export class DropDown {
     const options = this.#element.querySelectorAll('.list__item');
     const selected = this.#element.querySelector('.selected');
 
-    const ul = document.createElement('ul');
+    // const ul = document.createElement('ul');
+
+    // ul.classList.add('multiselectTag');
 
     options.forEach((option, index) => {
       option.addEventListener('click', (event) => {
         const item = this.#items[index];
 
-        const li = document.createElement('li');
-        const btn = document.createElement('button');
-        const text = document.createTextNode('X');
-
-        // let textLi = document.createTextNode(item);
-        // t.toString();
-        // console.log(textLi);
-
-        btn.appendChild(text);
-        btn.addEventListener('click', () => {
-          console.log('aaaa');
-          ul.removeChild(li);
-        });
-
-        ul.classList.add('multiselectTag');
-        console.log(ul);
-
         if (multiselect) {
           event.stopPropagation();
           option.classList.toggle('active');
@@ -276,24 +261,26 @@ export class DropDown {
 
             const checkIndex = this.#indexes.indexOf(index);
 
+            let templete = '';
             if (checkIndex === -1) {
               this.#indexes.push(index);
 
               if (this.#checkItemStruct(item)) {
                 this.#value.push(item.title);
               } else {
-                let textLi = document.createTextNode(item);
-                li.appendChild(textLi);
-                li.appendChild(btn);
-                ul.appendChild(li);
-
                 this.#value.push(item);
               }
 
-              // selected.innerText = this.#value;
+              //TODO refactoring code!!!!
+              if (multiselectTag) {
+                for (let i = 0; i < this.#value.length; i++) {
+                  templete += `<li>${this.#value[i]}<button>X</button></li>`;
+                }
 
-              selected.innerText = '';
-              selected.appendChild(ul);
+                selected.innerHTML = `<ul class="multiselectTag">${templete}</ul>`;
+              } else {
+                selected.innerText = this.#value;
+              }
 
               return;
             }
@@ -301,10 +288,22 @@ export class DropDown {
             this.#indexes.splice(checkIndex, 1);
             this.#value.splice(checkIndex, 1);
 
+            if (multiselectTag) {
+              for (let i = 0; i < this.#value.length; i++) {
+                templete += `<li>${this.#value[i]} <button>X</button></li>`;
+              }
+
+              selected.innerHTML = `<ul class="multiselectTag">${templete}</ul>`;
+            }
+
             if (!this.#value.length) {
               selected.innerText = placeholder;
             } else {
-              selected.innerText = this.#value;
+              if (multiselectTag) {
+                selected.innerHTML = `<ul class="multiselectTag">${templete}</ul>`;
+              } else {
+                selected.innerText = this.#value;
+              }
             }
           }
         } else {
diff --git a/src/index.js b/src/index.js
index 4ab2beb..eaa4547 100644
--- a/src/index.js
+++ b/src/index.js
@@ -45,10 +45,10 @@ const dropdown2 = new DropDown({
 });
 
 setTimeout(() => {
-  console.log(dropdown2.value);
+  console.log(dropdown.value);
 }, 10000);
 setTimeout(() => {
-  console.log(dropdown2.indexes);
+  console.log(dropdown.indexes);
 }, 10000);
 
 //ToDo: paste the desired url;