<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Thanks.<br>
<br>
Yeah I figured it was more just being sloppy than deceptive. Just
getting the vibe :)<br>
<br>
Paul<br>
<br>
<br>
<br>
On 20/02/2012 3:15 PM, Emil Uzelac wrote:
<blockquote
cite="mid:CAG4s18hAdDVjRgdgxa5fOPpN0LXV0+9wETE7XF1Tx6fJRGMUvA@mail.gmail.com"
type="cite"><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
moz-do-not-send="true"
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 moz-do-not-send="true"
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>
_______________________________________________<br>
theme-reviewers mailing list<br>
<a moz-do-not-send="true"
href="mailto:theme-reviewers@lists.wordpress.org"
target="_blank">theme-reviewers@lists.wordpress.org</a><br>
<a moz-do-not-send="true"
href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers"
target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
</blockquote>
</div>
<br>
</div>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
theme-reviewers mailing list
<a class="moz-txt-link-abbreviated" href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a>
<a class="moz-txt-link-freetext" href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a>
</pre>
</blockquote>
</body>
</html>