[theme-reviewers] FB Like button
Paul Appleyard
paul at spacecat.com
Mon Feb 20 05:40:33 UTC 2012
Thanks.
Yeah I figured it was more just being sloppy than deceptive. Just
getting the vibe :)
Paul
On 20/02/2012 3:15 PM, Emil Uzelac wrote:
> Paul,
>
> Not sure if this is "out-and-out evil" and if something like this
> should be pushed over to plugin.
>
> 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
> http://developers.facebook.com/docs/guides/web/. This was a
> straight-up paste and nothing has been changed from what I could see.
>
> Should that be in Theme Options maybe, well it could.
>
> Emil
>
> On Sun, Feb 19, 2012 at 10:58 PM, Paul Appleyard <paul at spacecat.com
> <mailto:paul at spacecat.com>> wrote:
>
> 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.
>
> 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.
>
> Paul
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> <mailto:theme-reviewers at lists.wordpress.org>
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
>
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20120220/0d4f0a79/attachment.htm>
More information about the theme-reviewers
mailing list