[theme-reviewers] FB Like button

Paul Appleyard paul at spacecat.com
Mon Feb 20 05:40:33 UTC 2012


Yeah I figured it was more just being sloppy than deceptive. Just 
getting the vibe :)


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