javascript - Jquery matching multiple elements with same idclass - Stack Overflow

admin管理员组

文章数量:1279044

I am working on showing a message when a particular element has a width of less than 50, however there are multiple elements with the same class and only the first element in the page is displaying the message. here is the jsfiddle /

Here is the function.

function checkads() {
    if ($('#container').height() < 50) {
        $('#container').parent().parent().prepend('<div id="ad-notice">Please support our website</div>');
    //
}
}

$(document).ready(checkads);

My question is, how do you make the message prepend to all element id instances found. I have various advertisements around my website, they are all wrapped in an div element named advertisement_container then how do I match them all at once

I am working on showing a message when a particular element has a width of less than 50, however there are multiple elements with the same class and only the first element in the page is displaying the message. here is the jsfiddle http://jsfiddle/MaNdn/23/

Here is the function.

function checkads() {
    if ($('#container').height() < 50) {
        $('#container').parent().parent().prepend('<div id="ad-notice">Please support our website</div>');
    //
}
}

$(document).ready(checkads);

My question is, how do you make the message prepend to all element id instances found. I have various advertisements around my website, they are all wrapped in an div element named advertisement_container then how do I match them all at once

• javascript
• jquery

Share Improve this question Follow edited Jul 6, 2013 at 0:38 David Garcia asked Jul 6, 2013 at 0:33 David GarciaDavid Garcia 2,6961818 gold badges6161 silver badges9494 bronze badges 3

• 2 IDs are unique. Please do not use the same ID for multiple elements. – Benjamin Gruenbaum Commented Jul 6, 2013 at 0:34
• I have various advertisements around my website, they are all wrapped in an div element named advertisement_container then how do I match them all at once? – David Garcia Commented Jul 6, 2013 at 0:38
• 2 @JoseDavidGarciaLlanos use class instead – Dom Commented Jul 6, 2013 at 0:39

Add a ment  | 

2 Answers 2

Sorted by: Reset to default 7

You need to use each() to iterate through the matched element. Instead of using same id for multiple elements use the same class as the id of element is supposed to be unique. To select multiple elements with same class you can use attribute selector like $('[id=container]') but it is better to use class and keep the ids of elements unique.

Live Demo

function checkads() {
    $('.someclass').each(function(){           
       if($(this).height() < 50) {
             $(this).parent().parent().prepend('<div id="ad-notice">Please support our website</div>');
       }
   });
}

$(document).ready(checkads);

ID has to be unique. Change it to be a class, not an ID, then you can use filter and something like

function checkads() {
    $('.container').filter(function (index) {
        return $(this).height() < 50;
    }).parent().parent().prepend('<div id="ad-notice">Please support our website</div>');
}

本文标签： javascriptJquery matching multiple elements with same idclassStack Overflow