I just want to wrap a div with the slideshow class around every 3 divs with a class for search audio.
One error I got was that it said elem.parentElement is undefined...
So this part works well...
const audioBlocks = document.querySelectorAll('.search-audio'); const slider = document.querySelector('.slider'); const audioBlockArr = Array.from(audioBlocks); function groupBlocks(arr, len) { let groups = [], i = 0, n = arr.length; while (i < n) { groups.push(arr.slice(i, i += len)); } return groups; } newArr = [...groupBlocks(audioBlockArr, 3)];
Here's the problem - wrapping a div around a new array of grouped elements.
let wrap = (array) => { array.forEach((elem) => { let div = document.createElement('div'); div.classList.add('slide'); div.innerHTML = ''; elem.parentElement.insertBefore(div, elem); div.appendChild(elem); }); } wrap(newArr);
Full code:
const audioBlocks = document.querySelectorAll('.search-audio'); const slider = document.querySelector('.slider'); const audioBlockArr = Array.from(audioBlocks); function groupBlocks(arr, len) { let groups = [], i = 0, n = arr.length; while (i < n) { groups.push(arr.slice(i, i += len)); } return groups; } newArr = [...groupBlocks(audioBlockArr, 3)]; let wrap = (array) => { array.forEach((elem) => { let div = document.createElement('div'); div.classList.add('slide'); div.innerHTML = ''; elem.parentElement.insertBefore(div, elem); div.appendChild(elem); }); } wrap(newArr);
body { font-family: system-ui; background: #f06d06; color: white; text-align: center; } .search-audio { height: 12rem; width: 12rem; background: rgb(255, 153, 0); margin: .5rem; display: inline-block; } .slide { background-color: #555; }
<body> <div class="slider"> <div class="search-audio 1"></div> <div class="search-audio 2"></div> <div class="search-audio 3"></div> <div class="search-audio 4"></div> <div class="search-audio 5"></div> <div class="search-audio 6"></div> <div class="search-audio 7"></div> <div class="search-audio 8"></div> <div class="search-audio 9"></div> <div class="search-audio 10"></div> <div class="search-audio 11"></div> <div class="search-audio 12"></div> <div class="search-audio 13"></div> <div class="search-audio 14"></div> <div class="search-audio 15"></div> <div class="search-audio 16"></div> <div class="search-audio 17"></div> <div class="search-audio 18"></div> <div class="search-audio 19"></div> <div class="search-audio 20"></div> </div> </body> </html>
I think you're overcomplicating things a bit, you can apply the style of
.search-audio
to each class of.slide
by using:nth-child
a third div to use CSS.This code may be wrong, but is similar to:
Here are some links that may be helpful in learning how to use them:
The problem is that
elem
is not a single element, it is an array of 3 elements created bygroupBlocks()
. So there is noelem.parentElement
.Use
elem[0].parentElement
to get the common parent element of all elements. Then loop through them to append each element to a new div.I renamed
elem
togroup
in the code below to more correctly match what it contains.