skibblenybbles / pinterestplus Goto Github PK

.

First off, thanks a bunch for this! I am hoping to use it to avoid parsing multiple buttons in one go (a page will contain at least 20 items, each with a Pinterest button).

However, while I have the script working - PinterestPlus.pinit(); works fine - I cannot seem to get it focus on one particular element, and it is parsing them all with the performance impact I am hoping to avoid. I have tried:

PinterestPlus.pinit('class');
PinterestPlus.pinit('id');
PinterestPlus.pinit('div.id');
PinterestPlus.pinit('div.classname);
PinterestPlus.pinit(document.getElementById('id'));

where of course 'id' and 'class' are valid for the divs concerned.

How should I be defining the element?

Thanks!

Andy

Hi Mike,

First off I wanted to thank you for writing this library, it's exactly what I was looking for after a couple rather frustrating attempts in trying to use the code provided by Pinterest in ways that it's not suited for.

Secondly, as the title implies, I'm curious about what kind of licensing were you thinking about releasing this under? I'd like to make use of it on a project I'm working on, license permitting of course.

BR,
Joonas

Hey there,
Firstly thanks for providing such a helpful piece of code.

Today I noticed that Pinterest have moved some of their assets to a different location, so my buttons weren't rendering at all. I went through your javascript and replaced this:
url:window.location.protocol==="https:"?"https://assets.pinterest.com/pinit.html":"http://pinit-cdn.pinterest.com/pinit.html"

With this:
url:window.location.protocol==="https:"?"https://assets.pinterest.com/pinit.html":"http://assets.pinterest.com/pinit.html"

I'm not sure if this is the best fix, but it works for now. Would like to know what you think

Cheers