search

powerpak / hn-sidebar

Hacker News Sidebar for Chrome
https://chrome.google.com/webstore/detail/hacker-news-sidebar/ngljhffenbmdjobakjplnlbfkeabbpma
MIT License
22 stars 5 forks source link

Tries to load 3 files from the visited domain on every page load #2

Closed supersoju closed 10 years ago

supersoju commented 10 years ago

The plugin tries to load some news.ycombinator.com resources (s.gif, y18.gif and grayarrow.gif) from the viewed page's domain if the page doesn't have a HN discussion.

These can be viewed when looking in the Chrome JS console.

Example: http://www.google.com/randompage Has 3 404 errors in the console https://www.google.com/ no errors

powerpak commented 10 years ago

You're right, it was because HN's HTML was getting parsed by jQuery and instantiating the elements must have fetched the URL's. Pushed a fix.