<font color="#333333"><font><font face="arial,helvetica,sans-serif">Paul,</font></font></font><div><font color="#333333"><font><font face="arial,helvetica,sans-serif"><br></font></font></font></div><div><font color="#333333"><font><font face="arial,helvetica,sans-serif">Not sure if this is "out-and-out evil" and if something like this should be pushed over to plugin.</font></font></font></div>
<div><font color="#333333"><font><font face="arial,helvetica,sans-serif"><br></font></font></font></div><div><font color="#333333"><font><font face="arial,helvetica,sans-serif">What you got in loop-front.php is very much standard JS code for FB Like Button and it's also available on their dev page <a href="http://developers.facebook.com/docs/guides/web/">http://developers.facebook.com/docs/guides/web/</a>. This was a straight-up paste and nothing has been changed from what I could see.</font></font></font></div>
<div><font color="#333333"><font><font face="arial,helvetica,sans-serif"><br></font></font></font></div><div><font color="#333333"><font><font face="arial,helvetica,sans-serif">Should that be in Theme Options maybe, well it could.</font></font></font></div>
<div><font color="#333333"><font><font face="arial,helvetica,sans-serif"><br></font></font></font></div><div><font color="#333333"><font><font face="arial,helvetica,sans-serif">Emil<br></font></font></font><br><div class="gmail_quote">
On Sun, Feb 19, 2012 at 10:58 PM, Paul Appleyard <span dir="ltr"><<a href="mailto:paul@spacecat.com">paul@spacecat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Just after a point of order; an author has inserted Facebook LIKE button JS code into loop-front.php, but has his OWN appId in the code; obviously this is unacceptable, but is it out-and-out evil, calling for a complete blackballing of the author, public humiliation etc.<br>
<br>
This is my second review, and my impression is that this sort of functionality kind of belongs to plugins, and that there should at least be an option for the theme user to add their own app id.<br>
<br>
Paul<br>
______________________________<u></u>_________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.<u></u>wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/<u></u>mailman/listinfo/theme-<u></u>reviewers</a><br>
</blockquote></div><br></div>