<HTML><HEAD></HEAD>
<BODY dir=ltr>
<DIV dir=ltr>
<DIV style="FONT-FAMILY: 'Calibri'; COLOR: #000000; FONT-SIZE: 12pt">
<DIV>sharing and social buttons could be cosmetic specific for the theme</DIV>
<DIV> </DIV>
<DIV>google analytics is backend which is plugin territory yeah</DIV>
<DIV> </DIV>
<DIV> </DIV>
<DIV 
style="FONT-STYLE: normal; DISPLAY: inline; FONT-FAMILY: 'Calibri'; COLOR: #000000; FONT-SIZE: small; FONT-WEIGHT: normal; TEXT-DECORATION: none">
<DIV style="FONT: 10pt tahoma">
<DIV> </DIV>
<DIV style="BACKGROUND: #f5f5f5">
<DIV style="font-color: black"><B>From:</B> <A title=chip@chipbennett.net 
href="mailto:chip@chipbennett.net">Chip Bennett</A> </DIV>
<DIV><B>Sent:</B> Tuesday, June 11, 2013 4:22 PM</DIV>
<DIV><B>To:</B> <A title=theme-reviewers@lists.wordpress.org 
href="mailto:theme-reviewers@lists.wordpress.org">[theme-reviewers]</A> </DIV>
<DIV><B>Subject:</B> Re: [theme-reviewers] Questions on my first 
review</DIV></DIV></DIV>
<DIV> </DIV></DIV>
<DIV 
style="FONT-STYLE: normal; DISPLAY: inline; FONT-FAMILY: 'Calibri'; COLOR: #000000; FONT-SIZE: small; FONT-WEIGHT: normal; TEXT-DECORATION: none">
<DIV dir=ltr>Plugin territory is a not-approve condition.</DIV>
<DIV class=gmail_extra><BR><BR>
<DIV class=gmail_quote>On Tue, Jun 11, 2013 at 6:45 PM, Harish <SPAN 
dir=ltr><<A href="mailto:me@harishchouhan.com" 
target=_blank>me@harishchouhan.com</A>></SPAN> wrote:<BR>
<BLOCKQUOTE 
style="BORDER-LEFT: #ccc 1px solid; MARGIN: 0px 0px 0px 0.8ex; PADDING-LEFT: 1ex" 
class=gmail_quote>
  <DIV lang=EN-IN vlink="purple" link="blue">
  <DIV>
  <P class=MsoNormal><SPAN 
  style="FONT-FAMILY: 'Calibri','sans-serif'; COLOR: #1f497d; FONT-SIZE: 11pt">Hello 
  Chip,<U></U><U></U></SPAN></P>
  <P class=MsoNormal><SPAN 
  style="FONT-FAMILY: 'Calibri','sans-serif'; COLOR: #1f497d; FONT-SIZE: 11pt"><U></U><U></U></SPAN> </P>
  <P class=MsoNormal><SPAN 
  style="FONT-FAMILY: 'Calibri','sans-serif'; COLOR: #1f497d; FONT-SIZE: 11pt">There 
  are many themes in the repo that have Google Analytics integration that could 
  be activated from Theme Options. <U></U><U></U></SPAN></P>
  <P class=MsoNormal><SPAN 
  style="FONT-FAMILY: 'Calibri','sans-serif'; COLOR: #1f497d; FONT-SIZE: 11pt"><U></U><U></U></SPAN> </P>
  <P class=MsoNormal><SPAN 
  style="FONT-FAMILY: 'Calibri','sans-serif'; COLOR: #1f497d; FONT-SIZE: 11pt">Should 
  a theme be rejected because it offers options for Google Analytics and Sharing 
  or social buttons? <U></U><U></U></SPAN></P>
  <P class=MsoNormal><SPAN 
  style="FONT-FAMILY: 'Calibri','sans-serif'; COLOR: #1f497d; FONT-SIZE: 11pt"><U></U><U></U></SPAN> </P>
  <P class=MsoNormal><SPAN 
  style="FONT-FAMILY: 'Calibri','sans-serif'; COLOR: #404040; FONT-SIZE: 11pt"><U></U><U></U></SPAN> </P>
  <P style="LINE-HEIGHT: 150%" class=MsoNormal><SPAN 
  style="LINE-HEIGHT: 150%; FONT-FAMILY: 'Calibri','sans-serif'; COLOR: #404040; FONT-SIZE: 11pt">Regards,<U></U><U></U></SPAN></P>
  <P style="LINE-HEIGHT: 150%" class=MsoNormal><SPAN 
  style="LINE-HEIGHT: 150%; FONT-FAMILY: 'Calibri','sans-serif'; COLOR: #404040; FONT-SIZE: 11pt">Harish</SPAN><SPAN 
  style="LINE-HEIGHT: 150%; FONT-FAMILY: 'Calibri','sans-serif'; COLOR: #0d0d0d; FONT-SIZE: 10pt"> 
  <B><U></U><U></U></B></SPAN></P>
  <P class=MsoNormal><SPAN 
  style="FONT-FAMILY: 'Calibri','sans-serif'; COLOR: #1f497d; FONT-SIZE: 11pt"><U></U><U></U></SPAN> </P>
  <P class=MsoNormal><B><SPAN 
  style="FONT-FAMILY: 'Calibri','sans-serif'; FONT-SIZE: 11pt" 
  lang=EN-US>From:</SPAN></B><SPAN 
  style="FONT-FAMILY: 'Calibri','sans-serif'; FONT-SIZE: 11pt" lang=EN-US> 
  theme-reviewers [mailto:<A 
  href="mailto:theme-reviewers-bounces@lists.wordpress.org" 
  target=_blank>theme-reviewers-bounces@lists.wordpress.org</A>] <B>On Behalf Of 
  </B>Chip Bennett<BR><B>Sent:</B> Wed 12 June 13 04:13 AM<BR><B>To:</B> 
  [theme-reviewers]<BR><B>Subject:</B> Re: [theme-reviewers] Questions on my 
  first review<U></U><U></U></SPAN></P>
  <DIV>
  <DIV class=h5>
  <P class=MsoNormal><U></U><U></U> </P>
  <DIV>
  <P class=MsoNormal>Google Analytics and sharing buttons are Plugin 
  territory.<U></U><U></U></P></DIV>
  <DIV>
  <P style="MARGIN-BOTTOM: 12pt" class=MsoNormal><U></U><U></U> </P>
  <DIV>
  <P class=MsoNormal>On Tue, Jun 11, 2013 at 5:00 PM, Peter Kakoma <<A 
  href="mailto:kakomap@gmail.com" target=_blank>kakomap@gmail.com</A>> 
  wrote:<U></U><U></U></P>
  <BLOCKQUOTE 
  style="BORDER-BOTTOM: medium none; BORDER-LEFT: #cccccc 1pt solid; PADDING-BOTTOM: 0cm; PADDING-LEFT: 6pt; PADDING-RIGHT: 0cm; MARGIN-LEFT: 4.8pt; BORDER-TOP: medium none; MARGIN-RIGHT: 0cm; BORDER-RIGHT: medium none; PADDING-TOP: 0cm">
    <DIV>
    <P class=MsoNormal>Hi Abhik,<U></U><U></U></P>
    <DIV>
    <P class=MsoNormal>Thanks for the feedback; point 1 has been duly 
    resolved...another implementation I picked from the framework author. We do 
    agree on point 1. <U></U><U></U></P></DIV>
    <DIV>
    <P style="MARGIN-BOTTOM: 12pt" class=MsoNormal>Point 2 though, as far as I 
    know, isn't grounds to reject a theme. Plugin/non-plugin territory, to the 
    best of my knowledge, has been debated ad nauseam and the jury is still out. 
    <U></U><U></U></P></DIV></DIV>
    <DIV>
    <DIV>
    <DIV>
    <P style="MARGIN-BOTTOM: 12pt" class=MsoNormal><U></U><U></U> </P>
    <DIV>
    <P class=MsoNormal>On Tue, Jun 11, 2013 at 7:31 PM, Abhik Biswas <<A 
    href="mailto:abhik@itsabhik.com" target=_blank>abhik@itsabhik.com</A>> 
    wrote:<U></U><U></U></P>
    <BLOCKQUOTE 
    style="BORDER-BOTTOM: medium none; BORDER-LEFT: #cccccc 1pt solid; PADDING-BOTTOM: 0cm; PADDING-LEFT: 6pt; PADDING-RIGHT: 0cm; MARGIN-LEFT: 4.8pt; BORDER-TOP: medium none; MARGIN-RIGHT: 0cm; BORDER-RIGHT: medium none; PADDING-TOP: 0cm">
      <DIV>
      <DIV>
      <P class=MsoNormal><SPAN 
      style="FONT-FAMILY: 'Verdana','sans-serif'">Pete,<U></U><U></U></SPAN></P></DIV>
      <DIV>
      <P style="MARGIN-BOTTOM: 12pt" class=MsoNormal><SPAN 
      style="FONT-FAMILY: 'Verdana','sans-serif'">If I were reviewing your 
      theme, I'd have rejected it straight away due to two major 
      issues.<U></U><U></U></SPAN></P></DIV>
      <DIV>
      <P class=MsoNormal><SPAN style="FONT-FAMILY: 'Verdana','sans-serif'">1. 
      Images should be resized using 'add_image_size', no external resizer is 
      allowed.<U></U><U></U></SPAN></P></DIV>
      <DIV>
      <P style="MARGIN-BOTTOM: 12pt" class=MsoNormal><SPAN 
      style="FONT-FAMILY: 'Verdana','sans-serif'">2. Those social sharing 
      buttons and Google Analytics integration. In my opinion, they fall under 
      plugin territory.<U></U><U></U></SPAN></P></DIV>
      <DIV>
      <P class=MsoNormal><SPAN style="FONT-FAMILY: 'Verdana','sans-serif'">But, 
      it all depends on the reviewer.<U></U><U></U></SPAN></P></DIV>
      <DIV>
      <P class=MsoNormal><SPAN 
      style="FONT-FAMILY: 'Verdana','sans-serif'"><U></U><U></U></SPAN> </P></DIV></DIV>
      <DIV>
      <DIV>
      <DIV>
      <P style="MARGIN-BOTTOM: 12pt" class=MsoNormal><U></U><U></U> </P>
      <DIV>
      <P class=MsoNormal>On Tue, Jun 11, 2013 at 9:55 AM, Peter Kakoma <<A 
      href="mailto:kakomap@gmail.com" target=_blank>kakomap@gmail.com</A>> 
      wrote:<U></U><U></U></P>
      <BLOCKQUOTE 
      style="BORDER-BOTTOM: medium none; BORDER-LEFT: #cccccc 1pt solid; PADDING-BOTTOM: 0cm; PADDING-LEFT: 6pt; PADDING-RIGHT: 0cm; MARGIN-LEFT: 4.8pt; BORDER-TOP: medium none; MARGIN-RIGHT: 0cm; BORDER-RIGHT: medium none; PADDING-TOP: 0cm">
        <DIV>
        <P class=MsoNormal>Thanks Springer. I've added to the 
        discussion<U></U><U></U></P></DIV>
        <DIV>
        <DIV>
        <DIV>
        <P style="MARGIN-BOTTOM: 12pt" class=MsoNormal><U></U><U></U> </P>
        <DIV>
        <P class=MsoNormal>On Tue, Jun 11, 2013 at 7:11 AM, J.J. Springer <<A 
        href="mailto:jjgspringer@gmail.com" 
        target=_blank>jjgspringer@gmail.com</A>> wrote:<U></U><U></U></P>
        <BLOCKQUOTE 
        style="BORDER-BOTTOM: medium none; BORDER-LEFT: #cccccc 1pt solid; PADDING-BOTTOM: 0cm; PADDING-LEFT: 6pt; PADDING-RIGHT: 0cm; MARGIN-LEFT: 4.8pt; BORDER-TOP: medium none; MARGIN-RIGHT: 0cm; BORDER-RIGHT: medium none; PADDING-TOP: 0cm">
          <DIV>
          <DIV>
          <P style="MARGIN-BOTTOM: 12pt" class=MsoNormal>Peter, <BR>I have 
          updated your ticket with some other things that I found while I was 
          reviewing the theme. I hope that they are helpful. Your ReadMe does 
          have the proper credits, so thank you for that. :)  I just wasn't 
          sure what licenses were allowed.<BR><BR><U></U><U></U></P></DIV>
          <DIV>
          <P style="MARGIN-BOTTOM: 12pt" class=MsoNormal>Otto, <BR>Thanks for 
          the speedy response! No worries about closing the ticket. I'm new to 
          the process so any help is appreciated. 
          <BR><BR><U></U><U></U></P></DIV>
          <DIV>
          <P style="MARGIN-BOTTOM: 12pt" class=MsoNormal>Amy,<BR>Thanks for the 
          detailed answers. I just want to make sure that I do everything 
          right!<BR><BR><U></U><U></U></P></DIV>
          <DIV>
          <P style="MARGIN-BOTTOM: 12pt" class=MsoNormal>Thanks again everyone 
          for the help! Can't wait for the next one!<U></U><U></U></P></DIV>
          <DIV>
          <P class=MsoNormal><SPAN 
          style="COLOR: #888888">J.J.<U></U><U></U></SPAN></P></DIV></DIV>
          <DIV>
          <DIV>
          <DIV>
          <P style="MARGIN-BOTTOM: 12pt" 
class=MsoNormal><U></U><U></U> </P>
          <DIV>
          <P class=MsoNormal>On Mon, Jun 10, 2013 at 8:42 PM, Peter Kakoma 
          <<A href="mailto:kakomap@gmail.com" 
          target=_blank>kakomap@gmail.com</A>> wrote:<U></U><U></U></P>
          <BLOCKQUOTE 
          style="BORDER-BOTTOM: medium none; BORDER-LEFT: #cccccc 1pt solid; PADDING-BOTTOM: 0cm; PADDING-LEFT: 6pt; PADDING-RIGHT: 0cm; MARGIN-LEFT: 4.8pt; BORDER-TOP: medium none; MARGIN-RIGHT: 0cm; BORDER-RIGHT: medium none; PADDING-TOP: 0cm">
            <DIV>
            <P class=MsoNormal>Hi Otto,<U></U><U></U></P>
            <DIV>
            <P class=MsoNormal>The theme in question is mine; only issue it 
            turns out after re-reading all the tips here is the base_64 
            encoding. I customized a  very popular and very user-friendly 
            framework -the Slightly Modified Options Framework. Very big themes 
            use it.<BR>That l<SPAN 
            style="FONT-FAMILY: 'Arial','sans-serif'; COLOR: #444444; FONT-SIZE: 10.5pt">ine, 
            "100% safe - ignore theme check nag", which I know got you :-) was 
            added by the Framework author. I took his word for it...seeing as 
            the framework is used everywhere. I figured several themes in the 
            repository already use it. I'm very aware of the base64 encoding 
            rule;  <BR><BR>I've removed the option altogether; the encoding 
            was being used to allow users import/export/transfer their settings. 
            Will find another way of doing it...and will make changes to the 
            framework repository so hopefully, the author can clean that 
            up.</SPAN><U></U><U></U></P></DIV>
            <DIV>
            <P class=MsoNormal><SPAN 
            style="FONT-FAMILY: 'Arial','sans-serif'; COLOR: #444444; FONT-SIZE: 10.5pt">I've 
            re-uploaded the theme: </SPAN><A 
            href="http://themes.trac.wordpress.org/ticket/12964" 
            target=_blank>http://themes.trac.wordpress.org/ticket/12964</A>. 
            <U></U><U></U></P></DIV>
            <DIV>
            <P class=MsoNormal>Springer, if you can, please take a look at it 
            again. (Seeing as you've already downloaded it and started the 
            review :-) )<U></U><U></U></P></DIV>
            <DIV>
            <P class=MsoNormal><U></U><U></U> </P></DIV>
            <DIV>
            <P class=MsoNormal>The credit link leads to my page: http:<A 
            href="http://kakoma.ug" target=_blank>kakoma.ug</A>, which I'm 
            upgrading right now. I can put something else if the current 'under 
            construction' page is an issue<U></U><U></U></P></DIV>
            <DIV>
            <P class=MsoNormal><U></U><U></U> </P></DIV>
            <DIV>
            <P class=MsoNormal>Oh, and the 'read me', as Springer can attest, 
            contains proper credits<U></U><U></U></P></DIV></DIV>
            <DIV>
            <DIV>
            <DIV>
            <P style="MARGIN-BOTTOM: 12pt" 
            class=MsoNormal><U></U><U></U> </P>
            <DIV>
            <P class=MsoNormal>On Tue, Jun 11, 2013 at 5:27 AM, Otto <<A 
            href="mailto:otto@ottodestruct.com" 
            target=_blank>otto@ottodestruct.com</A>> wrote:<U></U><U></U></P>
            <BLOCKQUOTE 
            style="BORDER-BOTTOM: medium none; BORDER-LEFT: #cccccc 1pt solid; PADDING-BOTTOM: 0cm; PADDING-LEFT: 6pt; PADDING-RIGHT: 0cm; MARGIN-LEFT: 4.8pt; BORDER-TOP: medium none; MARGIN-RIGHT: 0cm; BORDER-RIGHT: medium none; PADDING-TOP: 0cm">
              <DIV>
              <P class=MsoNormal>On Mon, Jun 10, 2013 at 8:54 PM, J.J. Springer 
              <<A href="mailto:jjgspringer@gmail.com" 
              target=_blank>jjgspringer@gmail.com</A>> 
              wrote:<U></U><U></U></P></DIV>
              <DIV>
              <P style="MARGIN-BOTTOM: 12pt" class=MsoNormal>> 2. If I get 
              any error of any sort, I assume that means that the theme 
              is<BR>> automatically rejected. As soon as I run into an error, 
              do I stop and mark<BR>> it rejected or keep going and try to 
              find as many errors as possible? Is<BR>> there a point at which 
              I should just stop and send it back?<U></U><U></U></P></DIV>
              <P class=MsoNormal>This is up to you. Additionally, I apologize 
              for usurping your review<BR>and failing the theme, but that kind 
              of ticked me off a bit.<BR><BR>If you wish to do a full review and 
              give the theme author more tips<BR>and things that he needs to 
              check on, that's perfectly acceptable.<U></U><U></U></P>
              <DIV>
              <P style="MARGIN-BOTTOM: 12pt" class=MsoNormal><BR>> 3. The 
              theme that I'm reviewing is a child theme of twentytwelve 
              and<BR>> therefore doesn't have the "required" theme template 
              files (missing<BR>> comments.php). Are child themes allowed? Is 
              there some special way that I'm<BR>> supposed to be reviewing 
              them?<U></U><U></U></P></DIV>
              <P class=MsoNormal>Child themes won't pass theme check, obviously, 
              because of missing<BR>files and such. The review guidelines still 
              stand though, and the<BR>child+parent must pass, as a whole, sort 
              of thing. The parent is<BR>presumed to already have passed, since 
              it's in the directory anyway.<U></U><U></U></P>
              <DIV>
              <P style="MARGIN-BOTTOM: 12pt" class=MsoNormal><BR><BR>> 4. The 
              person that I'm reviewing used a lot framework stuff that's MIT 
              or<BR>> WTFPL licensed. Is that ok?<U></U><U></U></P></DIV>
              <P class=MsoNormal>Both of those are GPL-Compat, so they're 
              fine.<BR><SPAN 
              style="COLOR: #888888"><BR><BR>-Otto</SPAN><U></U><U></U></P>
              <DIV>
              <DIV>
              <P 
              class=MsoNormal>_______________________________________________<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><U></U><U></U></P></DIV></DIV></BLOCKQUOTE></DIV>
            <P class=MsoNormal><BR><BR clear=all><U></U><U></U></P>
            <DIV>
            <P class=MsoNormal><U></U><U></U> </P></DIV></DIV></DIV>
            <P class=MsoNormal><SPAN style="COLOR: #888888">-- <BR><A 
            href="http://www.urbanlegendkampala.com" 
            target=_blank>www.urbanlegendkampala.com</A></SPAN><U></U><U></U></P></DIV>
            <P style="MARGIN-BOTTOM: 12pt" 
            class=MsoNormal><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><U></U><U></U></P></BLOCKQUOTE></DIV>
          <P class=MsoNormal><U></U><U></U> </P></DIV></DIV></DIV>
          <P style="MARGIN-BOTTOM: 12pt" 
          class=MsoNormal><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><U></U><U></U></P></BLOCKQUOTE></DIV>
        <P class=MsoNormal><BR><BR clear=all><U></U><U></U></P>
        <DIV>
        <P class=MsoNormal><U></U><U></U> </P></DIV>
        <P class=MsoNormal>-- <BR><A href="http://www.urbanlegendkampala.com" 
        target=_blank>www.urbanlegendkampala.com</A><U></U><U></U></P></DIV></DIV></DIV>
        <P style="MARGIN-BOTTOM: 12pt" 
        class=MsoNormal><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><U></U><U></U></P></BLOCKQUOTE></DIV>
      <P class=MsoNormal><U></U><U></U> </P></DIV></DIV></DIV>
      <P style="MARGIN-BOTTOM: 12pt" 
      class=MsoNormal><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><U></U><U></U></P></BLOCKQUOTE></DIV>
    <P class=MsoNormal><BR><BR clear=all><U></U><U></U></P>
    <DIV>
    <P class=MsoNormal><U></U><U></U> </P></DIV>
    <P class=MsoNormal>-- <BR><A href="http://www.urbanlegendkampala.com" 
    target=_blank>www.urbanlegendkampala.com</A><U></U><U></U></P></DIV></DIV></DIV>
    <P style="MARGIN-BOTTOM: 12pt" 
    class=MsoNormal><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><U></U><U></U></P></BLOCKQUOTE></DIV>
  <P 
  class=MsoNormal><U></U><U></U> </P></DIV></DIV></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>
<DIV> </DIV></DIV>
<P>
<HR>
_______________________________________________<BR>theme-reviewers mailing 
list<BR>theme-reviewers@lists.wordpress.org<BR>http://lists.wordpress.org/mailman/listinfo/theme-reviewers<BR></DIV></DIV></DIV></BODY></HTML>