I have a project using the Bulma CSS framework and raw Javascript. I need to remove the "is-pulled-right" class on mobile screen sizes, however only the first element responds to the script, and the others are ignored. What am I doing wrong?
window.addEventListener("resize", function () {
const classList = document.getElementById("isRight").classList
if(window.innerWidth < 820) {
classList.remove("is-pulled-right")
}
else{
classList.add("is-pulled-right");
}
});<link href="https://cdn.jsdelivr.net/npm/[email protected]/css/bulma.min.css" rel="stylesheet"/>
<section >
<div >
<div >
<div >
<h3 id="isRight" >First Title:</h3>
</div><!-- column left-->
<div >
<p>"Sed ut perspiciatis unde omnis iste natus error sit voluptatem accusantium doloremque laudantium, totam rem aperiam, eaque ipsa quae ab illo inventore veritatis et quasi architecto beatae vitae dicta sunt explicabo. Nemo enim ipsam voluptatem quia voluptas sit aspernatur aut odit aut fugit, sed quia consequuntur magni dolores eos qui ratione voluptatem sequi nesciunt. Neque porro quisquam est, qui dolorem ipsum quia dolor sit amet, consectetur, adipisci velit, sed quia non numquam eius modi tempora incidunt ut labore et dolore magnam aliquam quaerat voluptatem. Ut enim ad minima veniam, quis nostrum exercitationem ullam corporis suscipit laboriosam, nisi ut aliquid ex ea commodi consequatur? Quis autem vel eum iure reprehenderit qui in ea voluptate velit esse quam nihil molestiae consequatur, vel illum qui dolorem eum fugiat quo voluptas nulla pariatur?"</p>
</div><!-- column right -->
</div><!-- columns -->
</div><!-- container -->
</section>
<section >
<div >
<div >
<div >
<h3 id="isRight" >Second Title:</h3>
</div><!-- column left-->
<div >
<p>"Sed ut perspiciatis unde omnis iste natus error sit voluptatem accusantium doloremque laudantium, totam rem aperiam, eaque ipsa quae ab illo inventore veritatis et quasi architecto beatae vitae dicta sunt explicabo. Nemo enim ipsam voluptatem quia voluptas sit aspernatur aut odit aut fugit, sed quia consequuntur magni dolores eos qui ratione voluptatem sequi nesciunt. Neque porro quisquam est, qui dolorem ipsum quia dolor sit amet, consectetur, adipisci velit, sed quia non numquam eius modi tempora incidunt ut labore et dolore magnam aliquam quaerat voluptatem. Ut enim ad minima veniam, quis nostrum exercitationem ullam corporis suscipit laboriosam, nisi ut aliquid ex ea commodi consequatur? Quis autem vel eum iure reprehenderit qui in ea voluptate velit esse quam nihil molestiae consequatur, vel illum qui dolorem eum fugiat quo voluptas nulla pariatur?"</p>
</div><!-- column right -->
</div><!-- columns -->
</div><!-- container -->
</section>CodePudding user response：
1 - the id ("isRight") should be unic in html. you should use class instead of id .
2 - you can use document.querySelectorAll(".is-pulled-right") for select all element with is-pulled-right class instead of
document.getElementById("isRight")
const classList = document.querySelectorAll(".is-pulled-right")
window.addEventListener("resize", function () {
if(window.innerWidth < 700) {
classList.forEach( item => {
item.classList.remove("is-pulled-right")})
}
else{
console.log('more')
classList.forEach( item => {
item.classList.add("is-pulled-right")})
}
}); <link href="https://cdn.jsdelivr.net/npm/[email protected]/css/bulma.min.css" rel="stylesheet"/>
<section >
<div >
<div >
<div >
<h3 id="isRight" >First Title:</h3>
</div><!-- column left-->
<div >
<p>"Sed ut perspiciatis unde omnis iste natus error sit voluptatem accusantium doloremque laudantium, totam rem aperiam, eaque ipsa quae ab illo inventore veritatis et quasi architecto beatae vitae dicta sunt explicabo. Nemo enim ipsam voluptatem quia voluptas sit aspernatur aut odit aut fugit, sed quia consequuntur magni dolores eos qui ratione voluptatem sequi nesciunt. Neque porro quisquam est, qui dolorem ipsum quia dolor sit amet, consectetur, adipisci velit, sed quia non numquam eius modi tempora incidunt ut labore et dolore magnam aliquam quaerat voluptatem. Ut enim ad minima veniam, quis nostrum exercitationem ullam corporis suscipit laboriosam, nisi ut aliquid ex ea commodi consequatur? Quis autem vel eum iure reprehenderit qui in ea voluptate velit esse quam nihil molestiae consequatur, vel illum qui dolorem eum fugiat quo voluptas nulla pariatur?"</p>
</div><!-- column right -->
</div><!-- columns -->
</div><!-- container -->
</section>
<section >
<div >
<div >
<div >
<h3 id="isRight" >Second Title:</h3>
</div><!-- column left-->
<div >
<p>"Sed ut perspiciatis unde omnis iste natus error sit voluptatem accusantium doloremque laudantium, totam rem aperiam, eaque ipsa quae ab illo inventore veritatis et quasi architecto beatae vitae dicta sunt explicabo. Nemo enim ipsam voluptatem quia voluptas sit aspernatur aut odit aut fugit, sed quia consequuntur magni dolores eos qui ratione voluptatem sequi nesciunt. Neque porro quisquam est, qui dolorem ipsum quia dolor sit amet, consectetur, adipisci velit, sed quia non numquam eius modi tempora incidunt ut labore et dolore magnam aliquam quaerat voluptatem. Ut enim ad minima veniam, quis nostrum exercitationem ullam corporis suscipit laboriosam, nisi ut aliquid ex ea commodi consequatur? Quis autem vel eum iure reprehenderit qui in ea voluptate velit esse quam nihil molestiae consequatur, vel illum qui dolorem eum fugiat quo voluptas nulla pariatur?"</p>
</div><!-- column right -->
</div><!-- columns -->
</div><!-- container -->
</section>CodePudding user response：
the javascript file you can use eventlistener if u want to add more than 1 event on the window
const div = document.getElementById("isRight")
window.onresize = function(){
if(window.innerWidth < 820) {
div.classList.remove("is-pulled-right")
} else {
div.classList.add("is-pulled-right");
}
}
CodePudding user response：
Its is bad practice having multiple duplicates ids. since they should be unique. That's why document.getElementById(id); only returns one element.
Please use class instead. And then with something like https://developer.mozilla.org/en-US/docs/Web/API/Document/getElementsByClassName
If there is no way removing the ids then something like this should do it:
Array.from(document.querySelectorAll('#isRight')).map(e => e.classList).forEach(classlist => {
// your classlist logic
})