[theme-reviewers] [WordPress Themes] #8830: THEME: PageLines - 1.2.3

[Trent Lapinski]

Hey guys,

Was reviewing the PageLines theme when I came across a 3rd party image for a fallback image.

I wanted to confirm whether or not this is allowed by more seasoned reviewers? 

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?

Thanks!

--Trent Lapinski
=============
CEO of CyberChimps LLC
trent at cyberchimps.com
Mobile (714) 904-4280
Twitter @trentlapinski
http://CyberChimps.com

On Jul 31, 2012, at 5:43 PM, WordPress Themes <theme-reviewers at lists.wordpress.org> wrote:

> #8830: THEME: PageLines - 1.2.3
> ----------------------------+--------------------------
> Reporter:  arpowers         |       Owner:  applextrent
>    Type:  theme            |      Status:  assigned
> Priority:  major            |  Resolution:
> Keywords:  theme-pagelines  |
> ----------------------------+--------------------------
> 
> Comment (by applextrent):
> 
> Differential review only.
> 
> - For class.shortcodes.php what is the purpose of?
> http://placehold.it/1200x300&text=Specify+a+url+using+the+imageurl=&quot;&quot;+attribute!
> 
> "Themes are required to include within the Theme all images, scripts, and
> other bundled resources. Such resources must not be "hotlinked" from a
> third-party site."
> 
> I don't really see why you need to be loading a default image /
> placeholder that isn't part of the theme.
> 
> - Your screenshot does not match the defaults for your theme. The menu
> is not cropped correctly, the "Why You'll Love PageLines" does not even
> appear to be a part of the theme, and the three color icons do not match
> the black and white icons that come with the theme. I checked this latest
> version to see if you updated the icons to those colored ones but you
> haven't.
> 
> According to the review guidelines, Your screenshot "should be a
> "reasonable facsimile" of the Theme after it is initially activated with
> default options.'"
> 
> Please provide a more accurate screenshot of your theme in your next
> release.
> 
> Please wait while I consult with other team review members if I can
> approve this update or not.
> 
> -- 
> Ticket URL: <http://themes.trac.wordpress.org/ticket/8830#comment:2>
> WordPress Themes <http://themes.trac.wordpress.org/>
> WordPress.org Theme Directory Reviews

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20120731/22f32a6d/attachment.htm>