<font color="#333333"><font><font face="arial,helvetica,sans-serif">This would be definitely something we can discuss, placeholder such as </font></font></font><a href="http://placehold.it/1200x300&text=Specify+a+url+using+the+imageurl=%22%22+attribute!" target="_blank" style="font-family:arial,sans-serif;font-size:13px;color:rgb(17,85,204)">http://placehold.it</a> should't be something we need to forbid, service is pretty known and wide used by many developers around and they don't pose any treats to users. Well like Daniel said, just personal opinion that's all :)<div>
<font><font><br></font></font></div><div><font><font>Emil<br></font></font><br><div class="gmail_quote">On Tue, Jul 31, 2012 at 8:52 PM, Daniel Tara <span dir="ltr"><<a href="mailto:contact@onedesigns.com" target="_blank">contact@onedesigns.com</a>></span> wrote:<br>
<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>I don't see anything wrong with the screenshot of this theme. Other that the page scroll is a bit in offset and I believe that pretty easy to add to trivial to request a new screenshot. Just my opinion.</div>
<span class="HOEnZb"><font color="#888888"><div><br></div><div>Daniel</div></font></span><div><div class="h5"><br><div><div>On Aug 1, 2012, at 4:06 AM, Trent Lapinski wrote:</div><br><blockquote type="cite"><div style="word-wrap:break-word">
Thanks Otto. I've approved it and noted it for their next release.<div><br><div>
<span style="border-collapse:separate;border-spacing:0px"><span style="border-collapse:separate;font-family:Helvetica;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:-webkit-auto;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;font-size:medium"><div style="word-wrap:break-word">
<span style="border-collapse:separate;font-family:Helvetica;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:-webkit-auto;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;font-size:medium"><div style="word-wrap:break-word">
<span style="border-collapse:separate;font-family:Helvetica;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:-webkit-auto;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;font-size:medium"><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 6:00 PM, Otto <<a href="mailto:otto@ottodestruct.com" target="_blank">otto@ottodestruct.com</a>> wrote:</div><br><blockquote type="cite"><p><a href="http://Placehold.it/" target="_blank">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" <<a href="mailto:trent@cyberchimps.com" target="_blank">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=%22%22+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" target="_blank">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" target="_blank">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>
</blockquote></div><br></div></div>_______________________________________________<br>theme-reviewers mailing list<br><a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">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></blockquote></div><br></div></div></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></div>