<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#333333">
<font size="-1"><font face="Trebuchet MS">Previously approved themes
in general don't have big issues and in most cases found
problems are easy to fix, so I always leave a message in the
ticket. Then it depends to author's answer, if he'll have time
to fix and submit new version soon. Anyway they have to be fixed
soon or later if they are required (important). </font></font><br>
<br>
On 09.3.2012 12:42, Qamar Ashraf wrote:
<blockquote
cite="mid:CANYUyLdQugk_ANLvqms2DtMtj9H3P6cTt9HbFGXg1JiVyReBSw@mail.gmail.com"
type="cite">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 moz-do-not-send="true"
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
moz-do-not-send="true"
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 moz-do-not-send="true"
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
moz-do-not-send="true"
href="http://www.tutorialchip.com"
target="_blank">www.tutorialchip.com</a></font></div>
<div><font color="#666666"><a
moz-do-not-send="true"
href="http://www.freestockphotosclub.com"
target="_blank">www.freestockphotosclub.com</a></font></div>
<div><a moz-do-not-send="true"
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 moz-do-not-send="true"
href="mailto:theme-reviewers@lists.wordpress.org"
target="_blank">theme-reviewers@lists.wordpress.org</a><br>
<a moz-do-not-send="true"
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 moz-do-not-send="true"
href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
<a moz-do-not-send="true"
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 moz-do-not-send="true"
href="http://www.tutorialchip.com" target="_blank">www.tutorialchip.com</a></font></div>
<div><font color="#666666"><a moz-do-not-send="true"
href="http://www.freestockphotosclub.com" target="_blank">www.freestockphotosclub.com</a></font></div>
<div><a moz-do-not-send="true"
href="http://www.twunfollowapp.com" target="_blank">www.twunfollowapp.com</a></div>
<div><font color="#666666">@lifeobject1</font></div>
<br>
</div>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
theme-reviewers mailing list
<a class="moz-txt-link-abbreviated" href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a>
<a class="moz-txt-link-freetext" href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a>
</pre>
</blockquote>
<br>
<div class="moz-signature">-- <br>
<font size="2"><span style="font-family: Trebuchet MS, sans-serif">Yulian
Yordanov
<br>
<br>
<a class="moz-txt-link-abbreviated" href="mailto:yul.yordanov@gmail.com">yul.yordanov@gmail.com</a></span>
<br>
<a style="font-family: Trebuchet MS, sans-serif"
href="http://post-scriptum.info/">post-scriptum.info</a></font></div>
</body>
</html>