code: function loadnews() { document.getElementById('newshead').innerHTML = document.getElementById('newshead1').innerHTML; document.getElementById('newstext').innerHTML = document.getElementById('newstext1').innerHTML; document.getElementById('newsheadsub').innerHTML = document.getElementById('newshead2').innerHTML; } function swapnews() { if (document.getElementById('newshead').innerHTML = document.getElementById('newshead1').innerHTML) { document.getElementById('newshead').innerHTML = document.getElementById('newshead2').innerHTML; document.getElementById('newstext').innerHTML = document.getElementById('newstext2').innerHTML; document.getElementById('newsheadsub').innerHTML = document.getElementById('newshead1').innerHTML; } else { document.getElementById('newshead').innerHTML = document.getElementById('newshead1').innerHTML; document.getElementById('newstext').innerHTML = document.getElementById('newstext1').innerHTML; document.getElementById('newsheadsub').innerHTML = document.getElementById('newshead2').innerHTML; } } Code (markup): the body onload calls the "loadnews" function im trying to get it so that they can swap back and forth whenever the person clicks on the link containing the "swapnews" function so far it will only swap once, and wont swap back
common error, one i make every now and then too if (a = b) will SET a TO b, and return true if B wasn't null or undefined. if (a == b) will do what you want function loadnews() ..... function swapnews() { if (document.getElementById('newshead').innerHTML == document.getElementById('newshead1').innerHTML) { ..... } Code (markup):