<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Thanks Otto. I've approved it and noted it for their next release.<div><br><div apple-content-edited="true">
<span class="Apple-style-span" style="border-collapse: separate; border-spacing: 0px; "><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; "><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; "><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; "><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">--Trent Lapinski<br>=============<br>CEO of CyberChimps LLC<br><a href="mailto:trent@cyberchimps.com">trent@cyberchimps.com</a><br>Mobile (714) 904-4280<br>Twitter @trentlapinski<br>http://CyberChimps.com</div></span></div></span></div></span></span>
</div>
<br><div><div>On Jul 31, 2012, at 6:00 PM, Otto &lt;<a href="mailto:otto@ottodestruct.com">otto@ottodestruct.com</a>&gt; wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><p><a href="http://Placehold.it">Placehold.it</a> isn't objectionable enough to reject, but they should build it in for the next version. </p><p>Sorry for the brevity and typos, sent from my phone.</p>
<div class="gmail_quote">On Jul 31, 2012 7:53 PM, "Trent Lapinski" &lt;<a href="mailto:trent@cyberchimps.com">trent@cyberchimps.com</a>&gt; wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word"><div>Hey guys,</div><div><br></div><div>Was reviewing the PageLines theme when I came across a 3rd party image for a fallback image.</div><div><br></div><div>I wanted to confirm whether or not this is allowed by more seasoned reviewers?&nbsp;</div>
<div><br></div><div>Can I approve this version and let them address it in their next release or do I need to not-approve this and have them submit a new version without the the 3rd party image?</div><div><br></div><div>Thanks!</div>
<br><div>
<span style="text-indent:0px;letter-spacing:normal;font-variant:normal;text-align:-webkit-auto;font-style:normal;font-weight:normal;line-height:normal;border-collapse:separate;text-transform:none;font-size:medium;white-space:normal;font-family:Helvetica;word-spacing:0px"><span style="text-indent:0px;letter-spacing:normal;font-variant:normal;text-align:-webkit-auto;font-style:normal;font-weight:normal;line-height:normal;border-collapse:separate;text-transform:none;font-size:medium;white-space:normal;font-family:Helvetica;word-spacing:0px"><div style="word-wrap:break-word">
<span style="text-indent:0px;letter-spacing:normal;font-variant:normal;text-align:-webkit-auto;font-style:normal;font-weight:normal;line-height:normal;border-collapse:separate;text-transform:none;font-size:medium;white-space:normal;font-family:Helvetica;word-spacing:0px"><div style="word-wrap:break-word">
<span style="text-indent:0px;letter-spacing:normal;font-variant:normal;text-align:-webkit-auto;font-style:normal;font-weight:normal;line-height:normal;border-collapse:separate;text-transform:none;font-size:medium;white-space:normal;font-family:Helvetica;word-spacing:0px"><div style="word-wrap:break-word">
--Trent Lapinski<br>=============<br>CEO of CyberChimps LLC<br><a href="mailto:trent@cyberchimps.com" target="_blank">trent@cyberchimps.com</a><br>Mobile <a href="tel:%28714%29%20904-4280" value="+17149044280" target="_blank">(714) 904-4280</a><br>
Twitter @trentlapinski<br><a href="http://cyberchimps.com/" target="_blank">http://CyberChimps.com</a></div></span></div></span></div></span></span>
</div>
<br><div><div>On Jul 31, 2012, at 5:43 PM, WordPress Themes &lt;<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a>&gt; wrote:</div><br><blockquote type="cite">#8830: THEME: PageLines - 1.2.3<br>
----------------------------+--------------------------<br>Reporter: &nbsp;arpowers &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;| &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;Owner: &nbsp;applextrent<br> &nbsp;&nbsp;&nbsp;Type: &nbsp;theme &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;| &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;Status: &nbsp;assigned<br>Priority: &nbsp;major &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;| &nbsp;Resolution:<br>Keywords: &nbsp;theme-pagelines &nbsp;|<br>
----------------------------+--------------------------<br><br>Comment (by applextrent):<br><br> Differential review only.<br><br> - For class.shortcodes.php what is the purpose of?<br> <a href="http://placehold.it/1200x300&amp;text=Specify+a+url+using+the+imageurl=%22%22+attribute!" target="_blank">http://placehold.it/1200x300&amp;text=Specify+a+url+using+the+imageurl=&amp;quot;&amp;quot;+attribute!</a><br>
<br> "Themes are required to include within the Theme all images, scripts, and<br> other bundled resources. Such resources must not be "hotlinked" from a<br> third-party site."<br><br> I don't really see why you need to be loading a default image /<br>
 placeholder that isn't part of the theme.<br><br> - Your screenshot does not match the defaults for your theme. The menu<br> is not cropped correctly, the "Why You'll Love PageLines" does not even<br> appear to be a part of the theme, and the three color icons do not match<br>
 the black and white icons that come with the theme. I checked this latest<br> version to see if you updated the icons to those colored ones but you<br> haven't.<br><br> According to the review guidelines, Your screenshot "should be a<br>
 "reasonable facsimile" of the Theme after it is initially activated with<br> default options.'"<br><br> Please provide a more accurate screenshot of your theme in your next<br> release.<br><br> Please wait while I consult with other team review members if I can<br>
 approve this update or not.<br><br>-- <br>Ticket URL: &lt;<a href="http://themes.trac.wordpress.org/ticket/8830#comment:2" target="_blank">http://themes.trac.wordpress.org/ticket/8830#comment:2</a>&gt;<br>WordPress Themes &lt;<a href="http://themes.trac.wordpress.org/" target="_blank">http://themes.trac.wordpress.org/</a>&gt;<br>
<a href="http://wordpress.org/" target="_blank">WordPress.org</a> Theme Directory Reviews<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>theme-reviewers mailing list<br><a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>http://lists.wordpress.org/mailman/listinfo/theme-reviewers<br></blockquote></div><br></div></body></html>