Thanks Emil,<div>Please guide me in the following points as well.</div><div><ul><li>Let a theme is approved. I will definitely move into DIFF REVIEW only, if it has not major changes.</li><li>I usually see some other parts of code like header, footer, menus etc. So let find some Required or Recommended items are in need to address. So,</li>
<ul><li>Should i ask the reporter to fix it ?</li><li>Or, Approve the theme and leave recommendations for the next theme revision.</li></ul></ul><div><br></div><div><br></div><br><div class="gmail_quote">On Fri, Mar 9, 2012 at 3:34 PM, Emil Uzelac <span dir="ltr"><<a href="mailto:emil@themeid.com">emil@themeid.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><font color="#333333"><font><font face="arial,helvetica,sans-serif">If Theme was previously approved and if there are no major coding changes in functionalities or features, we only review the Diff. Grace period is up to reviewer, I like to give couple of days to ticket owner giving them a chance to fix the issues, re-submit Theme and of course note the last reviewed ticket in trac once the changes are ready.</font></font></font><div>
<font color="#333333"><font><font face="arial,helvetica,sans-serif"><br></font></font></font></div><div><font color="#333333"><font><font face="arial,helvetica,sans-serif">Theme in question did add support for header and background, however did not fulfill the requirements. (<i>Theme is recommended to incorporate the following WordPress core features, but is not required to do so. However, if incorporated, features must support the core WordPress implementation:</i>) this was left out from previous reviews, but what can we do now.</font></font></font></div>
<div><font color="#333333" face="arial, helvetica, sans-serif"><br></font></div><div><font color="#333333" face="arial, helvetica, sans-serif">I've seen cases (not many, but they exist) when author does a complete Theme overhaul each time new ticket was submitted, this is most definitely time to start from scratch, even in that case it would be nice to keep ticket open for couple of days, since it was already approved and in repository anyways. Same goes for all other major changes and that's why we have the Diff to see what was done since last review.</font></div>
<div><font color="#333333" face="arial, helvetica, sans-serif"><br></font></div><div><font color="#333333" face="arial, helvetica, sans-serif">I personally have zero tolerance for spam, sponsored links etc, but when author really tries hard and its here to truly contribute, we need to help them as much as we can.</font></div>
<div><font color="#333333"><font><font face="arial,helvetica,sans-serif"><br></font></font></font></div><div><font color="#333333"><font><font face="arial,helvetica,sans-serif">Cheers,</font></font></font></div><div><font color="#333333"><font><font face="arial,helvetica,sans-serif">Emil <br>
</font></font></font><br><div class="gmail_quote"><div><div class="h5">On Fri, Mar 9, 2012 at 4:05 AM, Qamar Ashraf <span dir="ltr"><<a href="mailto:life.object@gmail.com" target="_blank">life.object@gmail.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 class="h5">
I have just studied comments in my reviewed theme i.e<div><ul><li><a href="http://themes.trac.wordpress.org/ticket/6834" target="_blank">http://themes.trac.wordpress.org/ticket/6834</a></li></ul><div>I am writing here, because i am happy to be the part of co-operative and straight forward theme reviewers team. I also invite others to be the part of WPTRT, as i have learned a lot and still learning.</div>
<div><br></div><div>Once again thanks to, Emiluzelac and Chip Bennett.</div><div><br></div><div>One little question as well :)</div><div><ul><li>In which conditions, we can give a grace period to the theme reporter and how much ?</li>
</ul><div>Thanks</div></div><span><font color="#888888"><div><br></div>-- <br><font color="#666666">Regards</font><div><font color="#666666">Qamar Ashraf</font></div><div><font color="#666666">Project Manager</font></div>
<div><font color="#666666"><a href="http://www.tutorialchip.com" target="_blank">www.tutorialchip.com</a></font></div>
<div><font color="#666666"><a href="http://www.freestockphotosclub.com" target="_blank">www.freestockphotosclub.com</a></font></div><div><a href="http://www.twunfollowapp.com" target="_blank">www.twunfollowapp.com</a></div>
<div><font color="#666666">@lifeobject1</font></div><br>
</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">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><font color="#666666">Regards</font><div><font color="#666666">Qamar Ashraf</font></div><div><font color="#666666">Project Manager</font></div><div><font color="#666666"><a href="http://www.tutorialchip.com" target="_blank">www.tutorialchip.com</a></font></div>
<div><font color="#666666"><a href="http://www.freestockphotosclub.com" target="_blank">www.freestockphotosclub.com</a></font></div><div><a href="http://www.twunfollowapp.com" target="_blank">www.twunfollowapp.com</a></div>
<div><font color="#666666">@lifeobject1</font></div><br>
</div>