Hey Merci,<div><br></div><div>No worries. Just noticed the only single ticket in the #1 and pinged here and there. I asked here on purpose instead of discussing in the ticket as you have sent some notes but didn't assigned the ticket yourself.</div>
<div><br>All the best,<br clear="all"><br>Mario Peshev<br>Training and Consulting Services @ DevriX<br><a href="http://www.linkedin.com/in/mpeshev" target="_blank" class="vt-p">http://www.linkedin.com/in/mpeshev</a><br><a href="http://devrix.com" target="_blank" class="vt-p">http://devrix.com</a><br>
<a href="http://peshev.net/blog" target="_blank" class="vt-p">http://peshev.net/blog</a><br><br>
<br><br><div class="gmail_quote">On Tue, Dec 20, 2011 at 1:12 AM, Merci Javier <span dir="ltr"><<a href="mailto:mercijavier@gmail.com">mercijavier@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>Hi there,<br><br>Sorry I was incommunicado for a while. <br><br>I test BuddyPress themes up for review against current WP and BP versions. BP themes or BP child themes which may have been previously approved for older BP versions like 1.2+ (like most of the "buddypress" themes in WP theme repo) fail in current BP 1.5.2 version. So for me, review of BP themes submitted should be more than a diff review.<br>
<br>Custom Community theme passed for BP 1.5 where I focused on the BP template changes and missed the change in footer credits location. But even if there were only CSS changes submitted now, which is not the case for this theme, compatibility with the current WP 3.3 and BP 1.5.2 versions would require more than CSS changes to cover the points I brought up in the ticket alone without even starting the theme review proper:<br>
1. footer credits location<br>2. wp_enqueue_scripts() not wp_print_styles<br><br>While the second one is not yet included in our Theme Review guidelines, or I may have missed it, the styles did bleed through to the wp-admin area with a bp-default child theme which was activated when I upgraded to WP 3.3 - all orange links in dashboard.<br>
<br>So if you don't mind Mario, I am free now to take on the review of this ticket. Thanks.<br><br>Mercime<div><div></div><div class="h5"><br><br><div class="gmail_quote">On Mon, Dec 19, 2011 at 1:06 PM, Mario Peshev <span dir="ltr"><<a href="mailto:mario@peshev.net" target="_blank" class="vt-p">mario@peshev.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Not really, but since diff changes are CSS only and few very basic conditional statements, I don't see a reason not to be approved as an old theme.<br>
<br>Best regards,<div><br clear="all"><br>Mario Peshev<br>Training and Consulting Services @ DevriX<br>
<a href="http://www.linkedin.com/in/mpeshev" target="_blank" class="vt-p">http://www.linkedin.com/in/mpeshev</a><br><a href="http://devrix.com" target="_blank" class="vt-p">http://devrix.com</a><br><a href="http://peshev.net/blog" target="_blank" class="vt-p">http://peshev.net/blog</a><br>
<br>
<br><br></div><div><div></div><div><div class="gmail_quote">On Mon, Dec 19, 2011 at 11:04 PM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank" class="vt-p">chip@chipbennett.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
That one is a BuddyPress Theme. If you're comfortable with/able to<br>
review BP Themes, please feel free!<br>
<br>
Chip<br>
<div><div></div><div><br>
On 12/19/11, Mario Peshev <<a href="mailto:mario@peshev.net" target="_blank" class="vt-p">mario@peshev.net</a>> wrote:<br>
> mercime has commented on the #5800 ticket -<br>
> <a href="http://themes.trac.wordpress.org/ticket/5800" target="_blank" class="vt-p">http://themes.trac.wordpress.org/ticket/5800</a> - with some details that don't<br>
> lead to approval. From what I see he and Chip has approved the last 3<br>
> releases and my diff review found only style changes and very minor<br>
> customizations. I would personally mark this theme for approve, of course I<br>
> don't want to affect mercime's progress so far.<br>
><br>
> The theme is up and running, some styling has changed only, 5 days have<br>
> passed so far - ready for approval?<br>
><br>
> Mario Peshev<br>
> Training and Consulting Services @ DevriX<br>
> <a href="http://www.linkedin.com/in/mpeshev" target="_blank" class="vt-p">http://www.linkedin.com/in/mpeshev</a><br>
> <a href="http://devrix.com" target="_blank" class="vt-p">http://devrix.com</a><br>
> <a href="http://peshev.net/blog" target="_blank" class="vt-p">http://peshev.net/blog</a><br>
><br>
<br>
</div></div><font color="#888888">--<br>
Sent from my mobile device<br>
_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank" class="vt-p">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank" class="vt-p">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
</font></blockquote></div><br>
</div></div><br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank" class="vt-p">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank" class="vt-p">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" class="vt-p">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank" class="vt-p">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<br></blockquote></div><br></div>