<div dir="ltr">If a Theme already defines/uses sane defaults, what would be the purpose?</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Aug 4, 2014 at 6:49 AM, Greg Priday <span dir="ltr"><<a href="mailto:greg@siteorigin.com" target="_blank">greg@siteorigin.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">Would it be OK for a theme to flesh out a themes default settings specifically for the WordPress.org previewer?<div>
<br></div><div>Something like:</div><div><br></div><div>if( strpos( get_site_url(), '<a href="http://wp-themes.com/" target="_blank">http://wp-themes.com/</a>' ) !== false ) {</div>
<div>   // Add previewer defaults</div><div>}</div><div><br></div><div>My gut says this wouldn't be OK, so I've never done it. Still could be useful if used properly though.</div></div><div class="gmail_extra"><div>
<div class="h5"><br>
<br><div class="gmail_quote">On Mon, Aug 4, 2014 at 12:48 PM, Otto <span dir="ltr"><<a href="mailto:otto@ottodestruct.com" target="_blank">otto@ottodestruct.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">Also, you can point out that they won't have the ability to "configure" it in the WordPress.org previewer. So, there is that to consider. Having good default settings is important for many reasons.<span><font color="#888888"><div>



<br></div></font></span></div><div class="gmail_extra"><span><font color="#888888"><br clear="all"><div>-Otto</div></font></span><div><div>
<br><br><div class="gmail_quote">On Mon, Aug 4, 2014 at 5:44 AM, John Wilson <span dir="ltr"><<a href="mailto:mail@johnwilsononline.com" target="_blank">mail@johnwilsononline.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">Thanks Otto for clearing that up, will update my review accordingly and suggest to the developer.</div><div><div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Aug 4, 2014 at 11:37 AM, Otto <span dir="ltr"><<a href="mailto:otto@ottodestruct.com" target="_blank">otto@ottodestruct.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"><div>The terms "reasonable facsimile" seem to be throwing off a fair amount of reviewers from what I've read...<br>




</div><div><br></div><div>It's fine for a theme to have some setup and configuration involved to make it look right for the user, and for that type of thing to be reflected in the screenshot. The screenshot does not need to be *exactly* what you get after activating the theme for the first time, but more along the lines of the most common use.</div>






<div class="gmail_extra"><br></div><div class="gmail_extra">That said, if the theme looks like total crap after the initial install and activation, then that is something you probably want to tell the author too. The purpose of a review is not just to approve the theme for the directory, but to help the author to *improve* the theme overall. We want the themes in the directory to be good ones. If something is wrong with the theme, even if it's not going to be a showstopper against approval, then you should still speak up and give the author your advice. Work with them a bit. Suggest improvements, fixes, etc.</div>






<div class="gmail_extra"><br></div><div class="gmail_extra">And really, themes should use good default settings. Yes, a user will want to configure the theme, but those defaults determine what they see after activating the theme for the first time, so it should look pretty good there too. If the screenshot is extremely different from what the theme looks like from the start, then that's something the author might want to improve.</div>






<div class="gmail_extra"><br clear="all"><div>-Otto</div>
<div class="gmail_extra"><br></div><br><br><div class="gmail_quote"><div><div>On Mon, Aug 4, 2014 at 5:29 AM, John Wilson <span dir="ltr"><<a href="mailto:mail@johnwilsononline.com" target="_blank">mail@johnwilsononline.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">I'm conducting a theme review and one of the issues which has come up is screenshot.png<div>




<br></div>

<div>The dimensions are incorrect, and it isn't a “reasonable facsimile” of the Theme after it is initially activated with default options.</div>
<div><br></div><div>The developer has fixed the size, but raised a question. With the default theme after activation, the theme just looks like a blog. However, using the theme options they've created it's possible to create a theme which looks very different, and the whole purpose of a theme called 'Business Guru'.</div>







<div><br></div><div>Would it be acceptable to show what the theme looks like with the basic theme options applied, as somebody who wants this theme won't want it to look like a basic blog and may be put off by the screenshot?</div>







<div><br></div><div>Thanks,</div><span><font color="#888888"><div><div><br></div>-- <br><div dir="ltr"><div><b><br></b></div><b>John Wilson</b><br><div><br><br>Website: <a href="http://www.johnwilsononline.com/" target="_blank">http://www.johnwilsononline.com/</a><br>







Email: <a href="mailto:mail@johnwilsononline.com" target="_blank">mail@johnwilsononline.com</a><br>Twitter: <a href="http://twitter.com/johnwilsonuk" target="_blank">http://twitter.com/johnwilsonuk</a><br>LinkedIn: <a href="http://uk.linkedin.com/in/johnwilsonuk" target="_blank">http://uk.linkedin.com/in/johnwilsonuk</a></div>







</div>
</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></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><br clear="all"><div><br></div>-- <br><div dir="ltr"><div><b><br></b></div><b>John Wilson</b><br><div><br><br>Website: <a href="http://www.johnwilsononline.com/" target="_blank">http://www.johnwilsononline.com/</a><br>




Email: <a href="mailto:mail@johnwilsononline.com" target="_blank">mail@johnwilsononline.com</a><br>Twitter: <a href="http://twitter.com/johnwilsonuk" target="_blank">http://twitter.com/johnwilsonuk</a><br>LinkedIn: <a href="http://uk.linkedin.com/in/johnwilsonuk" target="_blank">http://uk.linkedin.com/in/johnwilsonuk</a></div>




</div>
</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" 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><br clear="all"><div><br></div>-- <br></div></div>I make free WordPress themes<br><a href="http://siteorigin.com" target="_blank">http://siteorigin.com</a>
</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>