<p>Placehold.it 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" <<a href="mailto:trent@cyberchimps.com">trent@cyberchimps.com</a>> 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? </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 <<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a>> wrote:</div><br><blockquote type="cite">#8830: THEME: PageLines - 1.2.3<br>
----------------------------+--------------------------<br>Reporter: arpowers | Owner: applextrent<br> Type: theme | Status: assigned<br>Priority: major | Resolution:<br>Keywords: theme-pagelines |<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&text=Specify+a+url+using+the+imageurl=""+attribute!" target="_blank">http://placehold.it/1200x300&text=Specify+a+url+using+the+imageurl=&quot;&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: <<a href="http://themes.trac.wordpress.org/ticket/8830#comment:2" target="_blank">http://themes.trac.wordpress.org/ticket/8830#comment:2</a>><br>WordPress Themes <<a href="http://themes.trac.wordpress.org/" target="_blank">http://themes.trac.wordpress.org/</a>><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>