Question

I’m having some trouble writing a function to change a background image on a div on document.ready

I haven’t made a jsfiddle as i think the problem is just my poor (but improving) jQuery skills. Please let me know if you think one is needed.

Background Info ->
I have a collection of div’s with a class of portlet-visible or portlet-hidden, each of these div’s will have another class of red-arrow (or a different color, but once i have one color it should be easy to extrapolate). When the page loads i would like a function that can find all divs with a class of portlet-hidden or portlet-visible and see if those have a class of red-arrow. If they do then change the background image src to a different value.

Im really struggling to work this one out, and any help is much appreciated.

My HTML

<div class="portlet-visible red-arrow"></div>

My CSS

div.portlet-visible
  {
    position:absolute;
    top:12px;
    right:10px;
    background-image:url(../images/red-arrow-up.png);
    width:14px;
    height:14px;    
  }

And finally my javascript

$(document).ready(function() {
        $(".portlet-hidden" && ".portlet-visible").each(function()  {
            if ($("this").hasClass(".red-arrow")) {
                $(this).css(background-image, url('"url(../images/blue-arrow-up.png)"')
            };
        });

    });     

Answer 1

Multiple selectors should be separated by a comma(,) and also css method takes a string or a map. Try this.

$(document).ready(function() {
        $(".portlet-hidden, .portlet-visible").each(function()  {
            if ($(this).hasClass("red-arrow")) {
                $(this).css('background-image', "url('../images/blue-arrow-up.png')")
            };
        });

    });