ansaurus

Question

Change all links that have an image in it to: javascript(alert(thehref));

Answer 1

+4 A:

for (var i= document.links.length; i-->0;) {
    if (document.links[i].getElementsByTagName('img').length!=0) {
        document.links[i].onclick= function() {
            alert(this.href);
        };
    }
}

encoded into a bookmarklet:

javascript:for%28i%3Ddocument.links.length%3Bi--%3E0%3B%29if%28document.links%5Bi%5D.getElementsByTagName%28%27img%27%29.length%21%3D0%29document.links%5Bi%5D.onclick%3Dfunction%28%29%7Balert%28this.href%29%3B%7D%3Bvoid%280%29%3B

bobince 2009-09-12 23:09:36

if you don't want to follow the link, add `return false` to the event handler; you might also want to move the function literal out of the loop for performance reasons (there's no need to create a new function object for each link: the event handler can easily be shared)

Christoph 2009-09-12 23:19:33

Answer 2

+1 A:

I would include jQuery in the external file, use noconflict mode, and do something like this:

$('a:has(img)').bind('click', function(event){
    event.preventDefault();
    alert($(this).children('img').get(0).attr('src'));
}

cpharmston 2009-09-12 23:10:26

ansaurus

tags:

views:

answers:

Change all links that have an image in it to: javascript(alert(thehref));

related questions