<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>