<div dir="ltr">Thank you all for the feedback! <div><br></div><div>And Justin, I definitely agree that there is a place for Easter eggs. In this case though, I'll recommend to the user that the ASCII-art in question be removed, as it is a bit too much self-promotion.</div>
<div><br></div><div>Best,</div><div>Ryan</div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Apr 3, 2014 at 8:15 PM, Edward Caissie <span dir="ltr"><<a href="mailto:edward.caissie@gmail.com" target="_blank">edward.caissie@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I agree with the "fun" aspect of adding "Easter egg" bits of code but in the case of the theme in question, I think they have crossed the line.</div>
<div class="gmail_extra"><span class="HOEnZb"><font color="#888888"><br clear="all">
<div>Edward Caissie<br>aka Cais.</div></font></span><div><div class="h5">
<br><br><div class="gmail_quote">On Thu, Apr 3, 2014 at 10:08 PM, Justin Tadlock <span dir="ltr"><<a href="mailto:justin@justintadlock.com" target="_blank">justin@justintadlock.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000">
I'm fine with fun things like ASCII-art as long as it's not publicly
visible in the HTML source like that. Don't do things like this for
brand promotion though. Do it for fun. And, put it in
functions.php or style.css.<br>
<br>
WP has a long history of Easter eggs. Even Kubrick has its own
thing (scroll to bottom):<br>
<a href="http://themes.svn.wordpress.org/default/1.7.2/style.css" target="_blank">http://themes.svn.wordpress.org/default/1.7.2/style.css</a><br>
<br>
I even dropped a quote from one of my favorite shows in my last
theme for fellow sci-fi nerds.<div><div><br>
<br>
<div>On 4/3/2014 7:37 PM, Chip Bennett
wrote:<br>
</div>
<blockquote type="cite">
<div dir="ltr">I would consider an ASCII-art HTML "powered by
datacube" comment to be tacky and inappropriate promotion.
AuthorURI is there, and equally publicly visible in source, in
style.css.</div>
<div class="gmail_extra">
<br>
<br>
<div class="gmail_quote">On Thu, Apr 3, 2014 at 8:32 PM, Emil
Uzelac <span dir="ltr"><<a href="mailto:emil@uzelac.me" target="_blank">emil@uzelac.me</a>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">
<div class="gmail_default" style="font-family:verdana,sans-serif">We don't have any
specific requirements, however I would treat the ASCII
like name="author", meaning it should not be in
header.php.</div>
<div class="gmail_default" style="font-family:verdana,sans-serif"><br>
</div>
</div>
<div class="gmail_extra"><br>
<br>
<div class="gmail_quote">
<div>
<div>On Thu, Apr 3, 2014 at 7:05 PM, Ryan
Cowles <span dir="ltr"><<a href="mailto:ryan.cowles@automattic.com" target="_blank">ryan.cowles@automattic.com</a>></span>
wrote:<br>
</div>
</div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>
<div>
<div dir="ltr">Howdy,
<div><br>
</div>
<div>I've been reviewing a theme that includes a
large ASCII art comment in header.php. I
imagine that this is not acceptable, but I
can't seem to tie it to a specific
rule/guideline. Here's the file for reference:</div>
<div><a href="https://themes.svn.wordpress.org/dc_genomics/1.0/header.php" target="_blank">https://themes.svn.wordpress.org/dc_genomics/1.0/header.php</a><br>
</div>
<div><br>
</div>
<div>Does this in fact violate any of the
requirements? And if so, which one?</div>
<div><br>
</div>
<div>Thanks!</div>
<span><font color="#888888">
<div>Ryan</div>
</font></span></div>
<br>
</div>
</div>
_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<br>
</blockquote>
</div>
<br>
</div>
<br>
_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<br>
</blockquote>
</div>
<br>
</div>
<br>
<fieldset></fieldset>
<br>
<pre>_______________________________________________
theme-reviewers mailing list
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a>
</pre>
</blockquote>
<br>
</div></div></div>
<br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<br></blockquote></div><br></div></div></div>
<br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<br></blockquote></div><br></div>