Hacker News Reader

Comment by Chramard | Hacker News Reader

This one transforms the links rather than opening them automatically. For pages with a lot of pdf links.

    javascript:(function(){dl=document.links;for(i=0;i<dl.length;++i){if(dl[i].href.substr(-4)==".pdf"){dl[i].href="http://docs.google.com/gview?embedded=true&url=+dl[i].href;}}})()

View on HN

Fixed a missing quote and added void(0) at the end:

  javascript:dl=document.links;for(i=0;i<dl.length;++i){if(dl[i].href.substr(-4)==".pdf"){dl[i].href="http://docs.google.com/gview?embedded=true&url="+dl[i].href;}}void(0);

Without the lambda, you run the risk of overwriting dl in the global scope, which could cause another script to misbehave.

Reply on news.ycombinator.com