<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><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 apple-content-edited="true">
<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; "><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 5:43 PM, WordPress Themes <<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a>> wrote:</div><br class="Apple-interchange-newline"><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=&quot;&quot;+attribute!">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">http://themes.trac.wordpress.org/ticket/8830#comment:2</a>><br>WordPress Themes <<a href="http://themes.trac.wordpress.org/">http://themes.trac.wordpress.org/</a>><br><a href="http://WordPress.org">WordPress.org</a> Theme Directory Reviews<br></blockquote></div><br></body></html>