joethei / obsidian-link-favicon

See the favicon for a linked website.
GNU General Public License v3.0
89 stars 2 forks source link

Too many console logs #42

Closed NomarCub closed 1 year ago

NomarCub commented 1 year ago

I have basically every website in my note logged to console, tens of log messages just by scrolling a little. They clog up the console if I try to debug other things. It's this line here: https://github.com/joethei/obsidian-link-favicon/blob/master/src/IconAdder.ts#L30 I should mention the comment above suggests this should only happen if the link has no protocol, but the links are all proper https:// links from big websites, so something seems off.

joethei commented 1 year ago

ooops, forgot to remove that one.

The comment is for the line below

joethei commented 1 year ago

will remove it in the next version, probably tomorrow or later today

NomarCub commented 1 year ago

I also found this typo in the readme: .external-link.external-link.external-link