<div dir="ltr">As Cais mentioned the screenshot alone is not acceptable. As far as namespace goes I can see why you would want to use it but that limits a lot of people seeing that is a PHP5.3+ feature.</div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Wed, May 22, 2013 at 1:02 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"><div><div>Yes, it does appear to be a challenging review ...<br><br></div>... <?php namespace theme; ?> will not work as a textdomain item. As Simon noted it is more a PHP item (IIRC, it is for establishing a namespace for the software which would work in favor of reducing/eliminating clashes with other PHP functions, etc.)<br>


<br></div>Also to note, 'theme' is not a very good textdomain to begin with ... the theme slug would be more appropriate; and, the logo as a screenshot is typically not acceptable either.<span class="HOEnZb"><font color="#888888"><br>
</font></span></div><div class="gmail_extra"><span class="HOEnZb"><font color="#888888">

<br clear="all"><div>Edward Caissie<br>aka Cais.</div>
<br><br></font></span><div class="gmail_quote"><div class="im">On Wed, May 22, 2013 at 3:46 PM, esmi at quirm dot net <span dir="ltr"><<a href="mailto:esmi@quirm.net" target="_blank">esmi@quirm.net</a>></span> wrote:<br>
</div><div><div class="h5"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<<a href="https://themes.trac.wordpress.org/ticket/12206" target="_blank">https://themes.trac.<u></u>wordpress.org/ticket/12206</a>><br>
<br>
Is using <?php namespace theme; ?> a reasonable excuse for then using 'theme' as a textdomain? My gut instinct says "no".<br>
<br>
That said, this theme is totally unlike any standard WPORG theme. The whole thing is pure actions hooks & filters (which is cool) but...<br>
<br>
1. functions are not being named. - they're being added (wholesale) as the second argument to add_action and add_filter.<br>
2. locate_template() is being used instead of get_template().<br>
<br>
In general, trying to apply the guidelines to this theme is like trying to plait fog. It looks more like it's intended as a framework than a theme. Anyone care to weigh in?<br>
<br>
Mel<span><font color="#888888"><br>
-- <br>
<a href="http://quirm.net" target="_blank">http://quirm.net</a><br>
<a href="http://blackwidows.co.uk" target="_blank">http://blackwidows.co.uk</a><br>
______________________________<u></u>_________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.<u></u>wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/<u></u>mailman/listinfo/theme-<u></u>reviewers</a><br>
</font></span></blockquote></div></div></div><br></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>