Guys, how can I improve the style of this chaining? document.querySelectorAll('ul[data-tabs]')

Question

Guys, how can I improve the style of this chaining? document.querySelectorAll('ul[data-tabs]')

.forEach((tabs) => {
tabs.querySelector('li').classList.add('tab-selected')
})

document.querySelectorAll('div[data-tabs-content]')
.forEach((tabsContent) => {
tabsContent.querySelectorAll('div:not(:first-child)')
.forEach((tabContent) => {
tabContent.style.display = 'none';
})
})

I can insert the foreach with the queryselector but then the line becomes a bit long. Also, I can save the query selector in a variable, but I don't need to cache it and it would be just useless. Is it fine like that or there are better ways to stylize it?

0

24.02.2020

1 ответов

17 просмотров

Wojciech Pawlik · Accepted Answer

Wojciech Pawlik

You can collapse nested query selectors into 1

0

24.02.2020

Похожие чаты

Guys, how can I improve the style of this chaining? document.querySelectorAll('ul[data-tabs]')

1 ответов

Похожие вопросы